[PATCH] [UBSan] Embed UBSan into ASan runtime (compiler-rt part).

Kostya Serebryany kcc at google.com
Mon Mar 30 11:16:24 PDT 2015


================
Comment at: lib/ubsan/ubsan_init.cc:52
@@ +51,3 @@
+  SpinMutexLock l(&ubsan_init_mu);
+  CHECK_NE(UBSAN_MODE_PLUGIN, ubsan_mode);
+  if (ubsan_mode == UBSAN_MODE_UNKNOWN)
----------------
ygribov wrote:
> Note that this will probably break GCC's dynamic UBSan runtime (e.g. when one part of the app is A-sanitized and another is UB-sanitized).
It may, and we will need to do something with this, most likely forbid some of the combination. 
The more combinations we support the more complex the code is, and we can not afford that.

http://reviews.llvm.org/D8646

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






More information about the llvm-commits mailing list