[libcxx-commits] [PATCH] D112727: [libc++] Remove Clang-11 support.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Oct 28 09:15:18 PDT 2021


Mordante added a comment.

In D112727#3093829 <https://reviews.llvm.org/D112727#3093829>, @ldionne wrote:

> 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 :-).

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 :-)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112727/new/

https://reviews.llvm.org/D112727



More information about the libcxx-commits mailing list