broadwaylamb planned changes to this revision.
broadwaylamb added a comment.

In D72717#1848509 <https://reviews.llvm.org/D72717#1848509>, @ldionne wrote:

> It's weird to mix Clang and libc++ test options because they test 
> fundamentally different things. But I don't mind about this patch since it 
> doesn't touch libc++ directly.


I understand that. This was supposed to be a temporary workaround.

I'm going to go with what @vvereschaka has proposed, although that will 
probably require some changes in libc++ CMake configuration.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72717



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to