[PATCH] D65480: [AMDGPU] Fix for vectorizer crash with pointers of different size

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 30 17:50:17 PDT 2019


rampitec added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:347-349
+  if (DL.getTypeStoreSizeInBits(PtrA->getType()) != PtrBitWidth ||
+      DL.getTypeStoreSizeInBits(PtrB->getType()) != PtrBitWidth)
+    return false;
----------------
arsenm wrote:
> Checking that PtrA and PtrB's address space are the same would be simpler 
The also need to be the same as original pointer address space before the strip.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65480/new/

https://reviews.llvm.org/D65480





More information about the llvm-commits mailing list