[PATCH] D77249: [MSan] Pass command line options to MSan with new pass manager

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 1 16:21:13 PDT 2020


nemanjai created this revision.
nemanjai added reviewers: eugenis, vitalybuka, philip.pfaffe, leonardchan, PowerPC.
Herald added projects: clang, Sanitizers.
Herald added subscribers: Sanitizers, cfe-commits.

There are a number of test cases that fail when clang is built to use NPM by default. This is due to a number of issues. One of those issues is that when we construct the memory sanitizer instrumentation pass with the NPM, we do not pass in the pertinent flags. This patch focuses on that issue.

For testing, this patch simply duplicates the tests that fail due to this issue and adds `-fexperimental-new-pass-manager` to the compile step. Once the NPM is the default, we can presumably remove those test cases.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D77249

Files:
  clang/lib/CodeGen/BackendUtil.cpp
  compiler-rt/test/msan/chained_origin_empty_stack_npm.cpp
  compiler-rt/test/msan/chained_origin_limits_npm.cpp
  compiler-rt/test/msan/chained_origin_memcpy_npm.cpp
  compiler-rt/test/msan/chained_origin_npm.cpp
  compiler-rt/test/msan/select_float_origin_npm.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77249.254329.patch
Type: text/x-patch
Size: 17882 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200401/fd88f5f4/attachment.bin>


More information about the llvm-commits mailing list