Triggered by commit r298620: "[LV] Vectorize GEPs".
If we encounter a vector GEP with scalar arguments, we splat the scalar
into a vector of appropriate size before we scatter the argument.
Paths
| Differential D31416
[Scalarizer] Handle scalar arguments in vector GEP ClosedPublic Authored by uabelho on Mar 28 2017, 12:15 AM.
Details Summary Triggered by commit r298620: "[LV] Vectorize GEPs". If we encounter a vector GEP with scalar arguments, we splat the scalar
Diff Detail Event TimelineComment Actions I don't know who should review this so I simply added the three guys that last made changes in the Scalarizer. Comment Actions Hi Mikael, r298620 was temporarily reverted because it exposed some other issues with vector GEP handling, but I plan to recommit it. Would you like for me to wait until after this patch is committed, or does it matter for your work? Comment Actions
Yes I saw and it doesn't matter, you can recommit when you want. I've done the Scalarizer changes locally in our repo since a few days anyway. This revision is now accepted and ready to land.Mar 30 2017, 10:36 AM Closed by commit rL299186: [Scalarizer] Handle scalar arguments in vector GEP (authored by uabelho). · Explain WhyMar 30 2017, 11:42 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 93219 lib/Transforms/Scalar/Scalarizer.cpp
test/Transforms/Scalarizer/vector-gep.ll
|
You should be able to drop the -o -