This will catch nasty Clang bugs like
https://bugs.llvm.org/show_bug.cgi?id=50592 before we merge stuff into
libc++ main.
Details
Details
- Reviewers
ldionne • Quuxplusone zoecarver Mordante phosek - Group Reviewers
Restricted Project - Commits
- rGcdb9d242deba: [libcxx][ci] enables assertions for runtimes-build
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
The change is fine by me, but I've some concerns with the build node used for this feature.
The Apple nodes aren't the fastest neither the most reliable build nodes we have.
This combination makes this node less than ideal for build iterations.
So I'm not happy to _only_ enable it for this node, but the improvements can be done in a separate patch.
@ldionne how do our Linux build nodes get their clang version and would it be easy to turn on the assertions for them?
Alternatively would it make sense to have one Linux node build clang HEAD with assertions to test C++2b?
I assume in the future would be nice to have a clang HEAD when we need compiler features to implement C++2b library features.