[PATCH] D104795: [IVDescriptors] Make pointer inductions compatible with opaque pointers
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 30 13:37:33 PDT 2021
nikic updated this revision to Diff 369539.
nikic edited the summary of this revision.
nikic added a comment.
Always use i8 element type for opaque pointers. I've come around to the idea that with opaque pointers, we should always use i8 GEPs for constant offsets. While in this particular case we can actually make a fairly good guess at a reasonable type to use, this is not possible in other cases, and for canonicalization reasons it's best to use the same approach everywhere.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104795/new/
https://reviews.llvm.org/D104795
Files:
llvm/include/llvm/Analysis/IVDescriptors.h
llvm/lib/Analysis/IVDescriptors.cpp
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
llvm/test/Transforms/LoopVectorize/opaque-ptr.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D104795.369539.patch
Type: text/x-patch
Size: 7977 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210830/000c70f1/attachment.bin>
More information about the llvm-commits
mailing list