Skip to content

Commit

Permalink
[llvm][ubsan] Inclusive language: replace use of blacklist HandleLLVM…
Browse files Browse the repository at this point in the history
…Options.cmake

This patch changes it to ignorelist and contains a filename change for the
.txt file that's called.

Reviewed By: vitalybuka

Differential Revision: https://reviews.llvm.org/D113689
  • Loading branch information
ZarkoT committed Nov 15, 2021
1 parent 3d01507 commit 44a64af
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions llvm/cmake/modules/HandleLLVMOptions.cmake
Expand Up @@ -870,9 +870,9 @@ if(LLVM_USE_SANITIZER)
append("-fsanitize=fuzzer-no-link" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
endif()
if (LLVM_USE_SANITIZER MATCHES ".*Undefined.*")
set(BLACKLIST_FILE "${CMAKE_SOURCE_DIR}/utils/sanitizers/ubsan_blacklist.txt")
if (EXISTS "${BLACKLIST_FILE}")
append("-fsanitize-blacklist=${BLACKLIST_FILE}"
set(IGNORELIST_FILE "${CMAKE_SOURCE_DIR}/utils/sanitizers/ubsan_ignorelist.txt")
if (EXISTS "${IGNORELIST_FILE}")
append("-fsanitize-ignorelist=${IGNORELIST_FILE}"
CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
endif()
endif()
Expand Down
@@ -1,4 +1,4 @@
# This blacklist should be applied when LLVM is built
# This ignorelist should be applied when LLVM is built
# with -fsanitize=undefined instrumentation. It exists
# because libstdc++ has some undefined behavior issues
# in some of the headers, in particular, stl_tree.h.
Expand Down

0 comments on commit 44a64af

Please sign in to comment.