[llvm] r367443 - [AMDGPU] Fix for vectorizer crash with pointers of different size

Arsenault, Matthew via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 06:14:46 PDT 2019




This is too aggressive, it will prevent any pointer pair that's addrspacecast'ed from ever being recognized as consecutive. What's the reason for not checking the size? I'm seeing regressions where pointers are getting loaded from memory and then casted to the right addresspace.

- Ben

Do you have an example where this matters? I would expect any cases that are vectorizable and involve an addrspacecast would be taken care of by InferAddressSpaces, so the vectorizer doesn’t need to worry about them.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190801/254af82b/attachment.html>


More information about the llvm-commits mailing list