[PATCH] D22535: [LSV] Don't assume that loads/stores appear in address order in the BB.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 19 14:19:01 PDT 2016


jlebar created this revision.
jlebar added a reviewer: asbirlea.
jlebar added subscribers: llvm-commits, arsenm.
Herald added a reviewer: tstellarAMD.
Herald added a subscriber: mzolotukhin.

getVectorizablePrefix previously didn't work properly in the face of
aliasing loads/stores.  It unwittingly assumed that the loads/stores
appeared in the BB in address order.  If they didn't, it would do the
wrong thing.

https://reviews.llvm.org/D22535

Files:
  lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
  test/Transforms/LoadStoreVectorizer/AMDGPU/insertion-point.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22535.64564.patch
Type: text/x-patch
Size: 6150 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160719/dc35726e/attachment.bin>


More information about the llvm-commits mailing list