[lldb][cmake] Also use local submodule visibility on Darwin
authorRaphael Isemann <teemperor@gmail.com>
Mon, 27 Apr 2020 12:58:50 +0000 (14:58 +0200)
committerRaphael Isemann <teemperor@gmail.com>
Tue, 28 Apr 2020 13:06:21 +0000 (15:06 +0200)
commit8baa0b9439b5788bc5a0a2ee45dfda01b7a5a43f
tree1bfa57ce05baeba508824c7be98edd94c14b4365
parent242e04ab27600b54e276e4bf5f38d118b0537fd4
[lldb][cmake] Also use local submodule visibility on Darwin

Summary:
Currently building LLVM on macOS and on other platforms with LLVM_ENABLE_MODULES is using different module flags,
which means that a passing modules build on macOS might fail on Linux and vice versa. -fmodules-local-submodule-visibility
is the mode that has clearer semantics and is closer to the actual C++ module standard, so let's make this the default everywhere.

We can still test building without local submodule visibility on an additional bot by just changing the respective CMake flag. However,
if building without local-submodule-visibility breaks we won't revert other commits and we won't loose LLDB's/Clang's test run
information.

Reviewers: aprantl, bruno, Bigcheese

Reviewed By: Bigcheese

Subscribers: abidh, dexonsmith, JDevlieghere, lldb-commits, mgorny, llvm-commits

Tags: #llvm, #lldb

Differential Revision: https://reviews.llvm.org/D74892
llvm/CMakeLists.txt