[PATCH] Sanitize printf functions

Kostya Serebryany kcc at google.com
Tue Jan 21 03:29:47 PST 2014



================
Comment at: lib/asan/lit_tests/TestCases/printf-5.c:4
@@ +3,3 @@
+// RUN: ASAN_OPTIONS=check_printf=0 %t 2>&1 | FileCheck --check-prefix=CHECK-OFF %s
+// RUN: %t 2>&1 | FileCheck --check-prefix=CHECK-OFF %s
+
----------------
Alexey Samsonov wrote:
> Once again, what's the point of testing the default configuration? In this way you'll have to update the tests on a flag flip, is this really what we want? Can we remove it, or at least have a single test for default configuration?
This was my suggestion, I think this *is* what we want. 


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



More information about the llvm-commits mailing list