-
Notifications
You must be signed in to change notification settings - Fork 257
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] clang frontend crashes on NDK 27.0.11718014 #2023
Comments
Reproduces in ToT clang from today (c54657887b) as well:
Reproducer fails with errors on r26. |
Non-regression and pretty late in the r27 cycle (and nothing available to cherry-pick yet), so planned for r28. We'll consider a cherry-pick into an r27 minor release if there's a low impact fix from upstream when the time comes. |
this is actually a regression, see llvm/llvm-project#93828. the crash shown by @pirama-arumuga-nainar is another crash. I replaced lines ranging from 85898 to 85912 with constexpr T&& operator [](uint i)&& noexcept
{
return move(Data[i]);
}
constexpr T const& operator [](uint i) const& noexcept
{
return Data[i];
}
constexpr T& operator [](uint i)& noexcept
{
return Data[i];
} and ndk 27 crashes while ndk 26 compiles. |
Oh, sorry about that, I misunderstood. Thanks for clarifying 👍 In that case, we're still pretty dependent on upstream to provide the fix, but if one becomes available we'll aim to do a minor release for it rather than only picking it up opportunistically (there are always exceptions if the fix is invasive and likely to do more harm than good, but that's very rare). |
upstream fixed: llvm/llvm-project@3d361b2 |
This is unlikely to make it to RC1. We can take it the next time we update the compiler in r27. (As we prep the next clang update, we also need to create a release branch for the r27 clang to be able to get new prebuilts.) |
(Thanks for proactively pursuing this in upstream and getting it fixed, btw!) |
+1. We're very appreciative of the work you've done to help us, here and on quite a few other bugs :) |
The fix for this bug was cherry-picked and will be part of r27 to be released later this month. |
Description
When using the NDK 27.0.11718014 the clang frontend crashes.
Following zip contains the Preprocessed source and associated run script:
ndk_frontend_crash.zip
Affected versions
r27
Canary version
No response
Host OS
Windows
Host OS version
Windows 10 and Windows 11
Affected ABIs
x86_64
Build system
CMake
Other build system
No response
minSdkVersion
23
Device API level
No response
The text was updated successfully, but these errors were encountered: