Fwd: [PATCH] Reorder shuffle and binary operation.

Serge Pavlov sepavloff at gmail.com
Mon Apr 28 00:18:16 PDT 2014


Hi all,

Strange but I don't see message from Phabricator, so forward it manually.
Could you please review the fix?

Thanks,
--Serge

------

This patch allows simplification:

    BinOp(shuffle(v1), shuffle(v2)) -> shuffle(BinOp(v1, v2))

if both shuffles use the same mask, and both shuffle within a single vector.

http://reviews.llvm.org/D3525

Files:
  lib/Transforms/InstCombine/InstCombine.h
  lib/Transforms/InstCombine/InstCombineAddSub.cpp
  lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
  lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
  lib/Transforms/InstCombine/InstCombineShifts.cpp
  lib/Transforms/InstCombine/InstCombineVectorOps.cpp
  lib/Transforms/InstCombine/InstructionCombining.cpp
  test/Transforms/InstCombine/vec_shuffle.ll
  test/Transforms/LoopVectorize/store-shuffle-bug.ll
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140428/c2482daf/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3525.8874.patch
Type: text/x-patch
Size: 16796 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140428/c2482daf/attachment.bin>


More information about the llvm-commits mailing list