[PATCH] D89915: [compiler-rt] Don't include libc++ headers from the source tree in MSAN

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 30 01:14:59 PDT 2020


phosek added inline comments.


================
Comment at: compiler-rt/lib/msan/tests/CMakeLists.txt:69
     DEPS gtest msan
-    CFLAGS ${MSAN_UNITTEST_INSTRUMENTED_CFLAGS} ${cflags}
+    CFLAGS -isystem ${CMAKE_CURRENT_BINARY_DIR}/../libcxx_msan_${arch}/include/c++/v1
+           ${MSAN_UNITTEST_INSTRUMENTED_CFLAGS} ${cflags}
----------------
phosek wrote:
> Could we add this to `MSAN_UNITTEST_COMMON_CFLAGS`?
Never mind, I see that the headers are per-arch now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89915



More information about the llvm-commits mailing list