[PATCH] D67345: [InstCombine] Allow values with multiple users in SimplifyDemandedVectorElts

Wed Oct 16 00:11:48 PDT 2019

piotr updated this revision to Diff 225165.
piotr added a comment.

As suggested by Nicolai, I extended the patch to also handle shufflevector users in addition to extractelement. The new generalized version also simplifies the case where three extract elements are used (the previous patch only handled two). The new code also makes it easy to extend the case list even more in the future.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D67345

Files:
  lib/Transforms/InstCombine/InstCombineInternal.h
  lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp
  lib/Transforms/InstCombine/InstCombineVectorOps.cpp
  test/Transforms/InstCombine/AMDGPU/amdgcn-demanded-vector-elts.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67345.225165.patch
Type: text/x-patch
Size: 13935 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191016/a448c3ea/attachment-0001.bin>