[PATCH] Add -fsanitize=leak to driver options.

Richard Smith richard at metafoo.co.uk
Thu May 23 15:58:38 PDT 2013



================
Comment at: test/Driver/fsanitize.c:59
@@ -53,2 +58,3 @@
+
 // RUN: %clang -target x86_64-linux-gnu -faddress-sanitizer -fthread-sanitizer -fno-rtti %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-ASAN-TSAN
 // CHECK-ASAN-TSAN: '-faddress-sanitizer' not allowed with '-fthread-sanitizer'
----------------
Alexey Samsonov wrote:
> Few more cases I can come up with:
> 1) All three of "-fsanitize=leak", "-fsanitize=address,leak", "-fsanitize=address,leak -fno-sanitize=address" work fine and
> resolve to correct "-fsanitize=" flag passed to frontend.
> 2) Standalone LSan runtime is linked in when you pass "-fsanitize=leak" and not linked in when you pass "-fsanitize=leak,address".
> This should go to test/Driver/sanitizer-ld.c
Please also test for "-fsanitize=leak,undefined" to make sure we don't pull in clang_rt.san in that case.


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



More information about the cfe-commits mailing list