[PATCH] D83921: [NewPM][ASan] Make ASan tests work under NPM

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/test/Instrumentation/AddressSanitizer/adaptive_global_redzones.ll:1
-; RUN: opt < %s -asan -asan-module -S | FileCheck %s
-; RUN: opt < %s -asan -asan-module -asan-mapping-scale=5 -S | FileCheck %s
+; RUN: opt < %s -asan -asan-module -enable-new-pm=0 -S | FileCheck %s
+; RUN: opt < %s -passes='require<asan-globals-md>,function(asan),module(asan-module)' -S | FileCheck %s
----------------
vitalybuka wrote:
> I assumed that "-asan -asan-module" is a explicit way to call old pm and -passes= for newpm
> should opt still use old pm for former even after switch to new PM by default? 
"-foo-pass" at some point run under NPM once we flip the flag, to avoid having to touch the vast majority of tests.
-enable-new-pm=0 pins it to legacy PM even after the NPM flip.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83921