[PATCH] [lsan] Convert all LSan tests to LIT tests. Remove the old testing interface.

Kostya Serebryany kcc at google.com
Thu May 23 06:45:50 PDT 2013



================
Comment at: lib/lsan/lit_tests/use_tls_static.cc:5
@@ +4,3 @@
+// RUN: --check-prefix=CHECK-leak %s
+// RUN: LSAN_OPTIONS=$LSAN_BASE:"use_tls=1" %t 2>&1 | FileCheck \
+// RUN: --check-prefix=CHECK-noleak %s
----------------
I don't think we require 80 chars in lit tests

================
Comment at: lib/lsan/lit_tests/use_tls_static.cc:2
@@ +1,3 @@
+// RUN: LSAN_BASE="report_blocks=1:use_stacks=0:use_registers=0"
+// RUN: %clangxx_lsan -m64 -O0 %s -o %t
+// RUN: LSAN_OPTIONS=$LSAN_BASE:"use_tls=0" %t 2>&1 | FileCheck \
----------------
maybe remove -m64 -O0 from there and move it into clangxx_lsan?


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



More information about the llvm-commits mailing list