[PATCH] D54848: [PM] Port LoadStoreVectorizer to the new pass manager.

Fri Nov 30 05:19:59 PST 2018

fedor.sergeev accepted this revision.
fedor.sergeev added a comment.
This revision is now accepted and ready to land.

LGTM.

================
Comment at: test/Transforms/LoadStoreVectorizer/X86/correct-order.ll:2
 ; RUN: opt -mtriple=x86_64-unknown-linux-gnu -load-store-vectorizer -S -o - %s | FileCheck %s
+; RUN: opt -mtriple=x86_64-unknown-linux-gnu -aa-pipeline=basic-aa -passes='function(load-store-vectorizer)' -S -o - %s | FileCheck %s

----------------
markus wrote:
> Unclear to my why -aa-pipeline=basic-aa is needed with the new PM (otherwise if not used AAResult contains no AAs and the test fails) while with the legacy PM -basicaa was not needed. Actually not clear to me if the two options are to be considered equivalent.
BasicAA is always available for function passes in legacy PM (see AAResultsWrapperPass::runOnFunction).
There is no unconditional AA analyses inserted into AA pipeline in new PM.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D54848