[PATCH] D29425: [SLP] Use SCEV to sort memory accesses

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 10:38:15 PST 2017


mkuper added inline comments.


================
Comment at: test/Transforms/SLPVectorizer/X86/jumbled-load.ll:8
 ; CHECK-LABEL: @jumbled-load(
-; CHECK-NEXT:    [[IN_ADDR:%.*]] = getelementptr inbounds i32, i32* %in, i64 0
+; CHECK-NEXT:    [[IN_ADDR:%.*]] = getelementptr inbounds i32, i32* [[IN:%.*]], i64 0
 ; CHECK-NEXT:    [[GEP_1:%.*]] = getelementptr inbounds i32, i32* [[IN_ADDR]], i64 3
----------------
mssimpso wrote:
> It looks like the changes in this test are unrelated to the patch. Probably left-overs from the update script?
Yes, it looks like the update script decided to pattern-match the first uses of function arguments instead of matching their names. :-S


https://reviews.llvm.org/D29425





More information about the llvm-commits mailing list