[PATCH] Add -fsanitize=leak to driver options.
Alexey Samsonov
samsonov at google.com
Mon May 27 03:49:52 PDT 2013
LGTM with minor comment.
================
Comment at: test/Driver/sanitizer-ld.c:184
@@ +183,3 @@
+// CHECK-LSAN-ASAN-LINUX: libclang_rt.asan-x86_64
+// CHECK-LSAN-ASAN-LINUX: "-lpthread"
+// CHECK-LSAN-ASAN-LINUX-NOT: "-lstdc++"
----------------
Here you test that lsan and asan are not added to ld invocation together.
Please use:
CHECK: ld
CHECK-NOT: libclang_rt.lsan // No lsan between "ld" and "asan"
CHECK: libclang_rt.asan
CHECK-NOT: libclang_rt.lsan // No lsan after "asan"
You may drop pthread and lstdc++
http://llvm-reviews.chandlerc.com/D837
BRANCH
feat/fsanitize_leak
ARCANIST PROJECT
clang
More information about the cfe-commits
mailing list