Since we no longer officially support Clang 11 remove the work-arounds
for this version.
Details
Details
- Reviewers
ldionne - Group Reviewers
Restricted Project - Commits
- rG7ee5e7e97c1c: [libc++] Remove Clang-11 support.
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Event Timeline
Comment Actions
LGTM!
Just FYI, to make sure you've removed all instances of Clang-11 related Lit features from the test suite, you can use this:
<build>/bin/llvm-lit libcxx/test libcxxabi/test --show-used-features
It will print a list of all the Lit features we mention in the test suite, and you can make sure that we don't mention clang-11 anymore after your patch. I guess grepping can also do the job, but sometimes it can be difficult to sort through the output if we mention clang-11 for things that are not Lit features. Anyway, just thought I'd let you know about this tool we have :-).
Comment Actions
Thanks I wasn't aware of that tool! I just grepped for everything that remotely resembled clang 11 and the number of hits was very manageable :-)