[PATCH] D39508: [ubsan] lit changes for lld testing, future lto testing.

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 28 14:34:02 PST 2017


eugenis added a comment.

Looks fine.
You need to update other sanitizers like asan to not add compile_wrapper the second time.



================
Comment at: test/lit.common.cfg:345
+config.clang = " " + " ".join(run_wrapper + [config.compile_wrapper, config.clang]) + " "
+config.target_cflags = " " + " ".join(target_cflags + extra_cflags) + " "
----------------
why not keep target_cflags as a list of strings? Looks like all users are joining them anyway.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D39508





More information about the llvm-commits mailing list