[PATCH] D48678: [InstCombine] enhance shuffle-of-binops to allow different variable ops (PR37806)

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 28 14:27:23 PDT 2018


spatel updated this revision to Diff 153394.
spatel added a comment.

Patch updated:

1. Rebased to trunk now that https://reviews.llvm.org/D48485 is committed (r335888).
2. Disallow the fold for any integer div/rem for now (see TODOs for follow-ups)
3. Added a comment about the safety of the shuffle mask.
4. Adjusted the tests - use 'xor' to demonstrate the multi-use limitation since we're not folding div/rem.


https://reviews.llvm.org/D48678

Files:
  lib/Transforms/InstCombine/InstCombineVectorOps.cpp
  test/Transforms/InstCombine/shuffle_select.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48678.153394.patch
Type: text/x-patch
Size: 14604 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180628/7776d616/attachment.bin>


More information about the llvm-commits mailing list