[PATCH] D21827: [CMake] Make -fmodules-local-submodule-visibility optional.

Adrian Prantl via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 10:22:33 PDT 2016


aprantl added a comment.

I forgot to mention that this patch also turns on the necessary "-fcxx-modules" on Darwin.

It's probably a good idea to ask more specific questions to the reviewers:

@beanz: Does my CMake copypasta make sense?
@bruno: Are you ok with this as an intermediate step on Darwin?
@rsmith: Let me know if there are any caveats from the Linux side — the intention is that this is NFC on Linux.


http://reviews.llvm.org/D21827





More information about the llvm-commits mailing list