<div dir="ltr">Hi all,<div><br></div><div>Strange but I don't see message from Phabricator, so forward it manually.</div><div>Could you please review the fix?<br></div><div><br></div><div>Thanks,</div><div>--Serge</div>
<div><br></div><div>------<br><br><div class="gmail_quote">This patch allows simplification:<br>
<br>
    BinOp(shuffle(v1), shuffle(v2)) -> shuffle(BinOp(v1, v2))<br>
<br>
if both shuffles use the same mask, and both shuffle within a single vector.<br>
<br>
<a href="http://reviews.llvm.org/D3525" target="_blank">http://reviews.llvm.org/D3525</a><br>
<br>
Files:<br>
  lib/Transforms/InstCombine/InstCombine.h<br>
  lib/Transforms/InstCombine/InstCombineAddSub.cpp<br>
  lib/Transforms/InstCombine/InstCombineAndOrXor.cpp<br>
  lib/Transforms/InstCombine/InstCombineMulDivRem.cpp<br>
  lib/Transforms/InstCombine/InstCombineShifts.cpp<br>
  lib/Transforms/InstCombine/InstCombineVectorOps.cpp<br>
  lib/Transforms/InstCombine/InstructionCombining.cpp<br>
  test/Transforms/InstCombine/vec_shuffle.ll<br>
  test/Transforms/LoopVectorize/store-shuffle-bug.ll<br>
</div><div><br></div>
</div></div>