[PATCH] D21934: Address two correctness issues in LoadStoreVectorizer

Alina Sbirlea via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 1 12:48:35 PDT 2016


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

GetBoundryInstruction returns the last instruction as the instruction which follows or end(). Otherwise the last instruction in the boundry set is not being tested by isVectorizable().
Partially solve reordering of instructions. More extensive solution to follow.

http://reviews.llvm.org/D21934

Files:
  lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
  test/Transforms/LoadStoreVectorizer/AMDGPU/interleaved-mayalias-store.ll
  test/Transforms/LoadStoreVectorizer/x86/preserve-order32.ll
  test/Transforms/LoadStoreVectorizer/x86/preserve-order64.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21934.62526.patch
Type: text/x-patch
Size: 4794 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160701/b537f82d/attachment.bin>


More information about the llvm-commits mailing list