[PATCH] D27466: [SLP] Fix sign-extends for type-shrinking

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 6 11:30:59 PST 2016


mkuper added inline comments.


================
Comment at: test/Transforms/SLPVectorizer/X86/minimum-sizes.ll:12
+; CHECK-LABEL: @PR31243
+; CHECK-NOT: sext i8 {{.*}} to i32
+;
----------------
mssimpso wrote:
> mkuper wrote:
> > Can you make the test more explicit by checking the extends we actually want to appear?
> Sure. I'll upload a new version.
Shouldn't we also have a test where we increase the bit-width because we don't know the sign bit?
I'm a bit surprised this didn't affect any of our existing tests.


https://reviews.llvm.org/D27466





More information about the llvm-commits mailing list