[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