[PATCH] D54848: [PM] Port LoadStoreVectorizer to the new pass manager.
Markus Lavin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 30 05:38:05 PST 2018
markus marked an inline comment as done.
markus added a comment.
Thanks for reviewing :)
================
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
----------------
fedor.sergeev wrote:
> 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.
In that case maybe we should have a -aa-pipeline=basic-aa option in all the .ll tests (I see that there are a few missing)? Not that it seems to matter for the tests passing but the pass has proven do depend on it for a few of them so might make sense.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D54848/new/
https://reviews.llvm.org/D54848
More information about the llvm-commits
mailing list