[PATCH] D26084: [lsan] [aarch64] Fix printing of pointers in make check tests - update

Kuba Brecka via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 10:31:11 PDT 2016


kubabrecka added inline comments.


================
Comment at: test/lsan/lit.common.cfg:34
 clang_cxxflags = config.cxx_mode_flags + clang_cflags
-clang_lsan_cflags = clang_cflags + lsan_cflags
-clang_lsan_cxxflags = clang_cxxflags + lsan_cflags
+lsan_incdir = config.test_source_root + "/../sanitizer_common/"
+clang_lsan_cflags = clang_cflags + lsan_cflags + ["-I%s" % lsan_incdir]
----------------
For consistency, this should also be just `/../` and the header should be included in LSan tests as `#include "sanitizer_common/print_address.h"`.


https://reviews.llvm.org/D26084





More information about the llvm-commits mailing list