[PATCH] D113689: [llvm][ubsan] Inclusive language: replace use of blacklist HandleLLVMOptions.cmake

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 15 10:43:16 PST 2021


bjope added inline comments.


================
Comment at: llvm/cmake/modules/HandleLLVMOptions.cmake:875
+    if (EXISTS "${IGNORELIST_FILE}")
+      append("-fsanitize-ignorelist=${IGNORELIST_FILE}"
              CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
----------------
Now I can't build llvm with ubsan, using an older version of clang, without getting lots of errors such as
  `clang-13: error: unknown argument: '-fsanitize-ignorelist=/workspace/llvm/llvm-main-sanitize-ubsan/llvm/utils/sanitizers/ubsan_ignorelist.txt'`
that seems a bit unfortunate (or are there different requirements on which compiler version that is needed when building llvm depending on if I enable sanitizers or not?).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113689



More information about the llvm-commits mailing list