[PATCH] Handle LLVM_USE_SANITIZER=Address; Undefined (and the other way around)

Alexey Samsonov vonosmas at gmail.com
Wed Feb 4 14:07:06 PST 2015


LGTM after fixing a nit below. We don't plan to support running UBSan with any other sanitizer except for ASan in the short term.


================
Comment at: cmake/modules/HandleLLVMOptions.cmake:401
@@ +400,3 @@
+      append_common_sanitizer_flags()
+      append("-fsanitize=address" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
+      append("-fsanitize=undefined -fno-sanitize=vptr,function -fno-sanitize-recover"
----------------
You can use a single append() statement:
  append("-fsanitize=address,undefined -fno-sanitize=...")

http://reviews.llvm.org/D7024

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list