================
@@ -371,6 +372,7 @@ LLVM <https://llvm.org/docs/BuildingADistribution.html>`_):
           -C /path/to/llvm-project/lldb/cmake/caches/Apple-lldb-macOS.cmake \
           -DLLVM_ENABLE_PROJECTS="clang;lldb" \
           -DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind" \
+          -DLLDB_ENFORCE_STRICT_TEST_REQUIREMENTS=ON \
----------------
JDevlieghere wrote:

I don't think we should add the flag here. If we wanted to make it the default, 
we could add it to the CMake cache (i.e. `Apple-lldb-macOS.cmake`) but I don't 
think it belongs there either. The motivation for the flag was to catch missing 
dependencies in CI. At desk, we want to streamline the development process. For 
example, if someone working on another LLVM subproject want to make sure their 
API change doesn't break building LLDB, they shouldn't be forced to install all 
the dependencies of our test suite. 

https://github.com/llvm/llvm-project/pull/130381
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to