[PATCH] D30211: [LV] Merge floating point and integer induction widening code

[Michael Kuperstein via Phabricator via llvm-commits]

mkuper added inline comments.


================
Comment at: test/Transforms/LoopVectorize/float-induction.ll:4
 ; RUN: opt < %s  -loop-vectorize -force-vector-interleave=2 -force-vector-width=1 -dce -instcombine -S | FileCheck --check-prefix VEC1_INTERL2 %s
+; RUN: opt < %s  -loop-vectorize -force-vector-interleave=1 -force-vector-width=2 -dce -simplifycfg -instcombine -S | FileCheck --check-prefix=VEC2_INTERL1_PRED_STORE %s
 
----------------
mssimpso wrote:
> mkuper wrote:
> > Did you run it through the update script? If you did, could you have the diff show the actual diff vs. running it with the old code?
> Hey Michael, can you clarify what you mean here? I did use the script to help generate the checks (then moved the test into this file). You're just wanting to see the lines that are different with and without the patch?
Yes, otherwise it's kind of hard to see what changed.
I suggest running the script over the test with the existing code, committing that test, and rebasing this patch on that. That way we can actually see what happened.


https://reviews.llvm.org/D30211