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
ci: upgrade LLVM to 18 #14148
ci: upgrade LLVM to 18 #14148
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #14148 +/- ##
==========================================
- Coverage 93.79% 93.23% -0.57%
==========================================
Files 2293 2206 -87
Lines 202903 192133 -10770
==========================================
- Hits 190317 179135 -11182
- Misses 12586 12998 +412 ☔ View full report in Codecov by Sentry. |
https://releases.llvm.org/18.1.0/projects/libcxx/docs/ReleaseNotes.html#build-system-changes This might be relevant
It seems like the failures have to do with throwing exceptions |
Running locally, getting things like you can't use try or throw when exceptions are disabled
|
I am not sure how that would affect the msan build. We only have one build that compiles with
I suggested turning exceptions off as a way to verify that hypothesis. When we turn exceptions off, nothing throws (we get more crashes). If that fixes the problem then there is something weird with exceptions. |
Blegh. Can we skip the generator just to test things? As in |
It seems like we also use |
This worked for me on my workstation. The tests did not pass before this change, and started to pass afterwards. I decided to use LLVM 18.1.1 because this is the version of Clang included with Fedora:40. I am a coward and fear upgrading the library beyond the corresponding version of the compiler.
FTR, it was related to exceptions: by default libc++ now compiles with the LLVM implementation of stack unwinding under exceptions. Meanwhile, the compiler on Fedora is (naturally) trying to use the "native" implementation of stack unwinding. The two things do not work together. |
#14076
This change is