[PATCH] D56538: [NewPM][TSan] Reiterate the TSan port

Philip Pfaffe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 11 02:08:11 PST 2019


philip.pfaffe marked 2 inline comments as done.
philip.pfaffe added inline comments.


================
Comment at: llvm/test/Instrumentation/ThreadSanitizer/tsan_basic.ll:2
 ; RUN: opt < %s -tsan -S | FileCheck %s
+; RUN: opt < %s -passes=tsan -S | FileCheck %s
 
----------------
vitalybuka wrote:
> what is about other tests?
I don't need to test tsan, just the pass pipeline specifics. The other tests appeared to mainly test tsan specifics, so I didn't touch them.

I can absolutely port the others, but that will (in theory at least) double the runtime. Do you have suggestions which other tests are meaningful to the pipeline?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56538/new/

https://reviews.llvm.org/D56538





More information about the llvm-commits mailing list