[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 23 02:59:12 PST 2018


markus added inline comments.


================
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
 
----------------
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.


Repository:
  rL LLVM

https://reviews.llvm.org/D54848





More information about the llvm-commits mailing list