[PATCH] D21934: Address two correctness issues in LoadStoreVectorizer

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 1 13:40:42 PDT 2016


arsenm added inline comments.

================
Comment at: test/Transforms/LoadStoreVectorizer/x86/preserve-order32.ll:1
@@ +1,2 @@
+; RUN: opt -mtriple=x86-linux -load-store-vectorizer -S -o - %s | FileCheck %s
+
----------------
jlebar wrote:
> Can we have a comment somewhere indicating what we're testing?  Same for the other test.  And maybe we should get rid of whatever instructions aren't relevant to the thing we're checking from our CHECKs?
Should this have -basicaa?


http://reviews.llvm.org/D21934





More information about the llvm-commits mailing list