[PATCH] D27990: [Thumb] Add support for tMUL in the compare instruction peephole optimizer

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 06:04:25 PST 2017


SjoerdMeijer updated this revision to Diff 83042.
SjoerdMeijer added a comment.

I have verified correctness and this patch does not show any problems in regression tests or benchmarks.  This peephole works on the vregs, so is still in SSA form. As a consequence, the movs won't redefine/kill the MUL operands which would make this reordering illegal. I have added an extra check though that the movs are not predicated. I have also changed the tests into .mir tests (and simplified the 2nd test).


https://reviews.llvm.org/D27990

Files:
  lib/Target/ARM/ARMBaseInstrInfo.cpp
  test/CodeGen/ARM/cmp1-peephole-thumb.mir
  test/CodeGen/ARM/cmp2-peephole-thumb.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27990.83042.patch
Type: text/x-patch
Size: 18644 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170104/8dbc7781/attachment.bin>


More information about the llvm-commits mailing list