[PATCH] D82915: [NewPM] Add -basic-aa to pr33196.ll

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 30 15:14:03 PDT 2020


aeubanks created this revision.
aeubanks added reviewers: fhahn, asbirlea.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The legacy pass manager implicitly adds BasicAA, but the new PM does
not. This causes pr33196.ll to fail under NPM.

There are almost certainly lots of other failures like this, wanted to
get some input on if adding -basic-aa to tests makes sense at scale.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D82915

Files:
  llvm/test/Transforms/NewGVN/pr33196.ll


Index: llvm/test/Transforms/NewGVN/pr33196.ll
===================================================================
--- llvm/test/Transforms/NewGVN/pr33196.ll
+++ llvm/test/Transforms/NewGVN/pr33196.ll
@@ -1,4 +1,4 @@
-; RUN: opt -S -newgvn %s | FileCheck %s
+; RUN: opt -S -basic-aa -newgvn %s | FileCheck %s
 
 ; CHECK: define i32 @main() {
 ; CHECK-NEXT: entry:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D82915.274621.patch
Type: text/x-patch
Size: 363 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200630/f1372a6a/attachment.bin>


More information about the llvm-commits mailing list