[PATCH] [lsan] CMakeLists and lit test configs for LeakSanitizer.

Sergey Matveev earthdok at google.com
Fri May 17 08:39:34 PDT 2013



================
Comment at: lib/lsan/CMakeLists.txt:5
@@ +4,3 @@
+  ${SANITIZER_COMMON_CFLAGS}
+  -fPIC
+  -I${COMPILER_RT_SOURCE_DIR}/lib
----------------
Alexander Potapenko wrote:
> Do you need -fPIC for your runtime?
I'm not sure how that got there. Removed.

================
Comment at: lib/lsan/tests/CMakeLists.txt:52
@@ +51,3 @@
+
+if(UNIX AND NOT APPLE
+    AND CAN_TARGET_x86_64
----------------
Alexander Potapenko wrote:
> Which systems belong to "UNIX AND NOT APPLE"?
Added a comment.

I copied this from TSan, so I probably don't need all of these conditions. Hopefully samsonov can shed some light on this.

================
Comment at: lib/lsan/tests/lit.site.cfg.in:9
@@ +8,3 @@
+  config.llvm_build_mode = config.llvm_build_mode % lit.params
+except KeyError,e:
+  key, = e.args
----------------
Alexander Potapenko wrote:
> missing space between "," and "e"
fixed


http://llvm-reviews.chandlerc.com/D804



More information about the llvm-commits mailing list