[PATCH] D34515: [ARM] Materialise some boolean values to avoid a branch

[Roger Ferrer Ibanez via Phabricator via llvm-commits]

rogfer01 updated this revision to Diff 117834.
rogfer01 marked 5 inline comments as done.
rogfer01 added a comment.

ChangeLog:

- Reordered combiner and added early exit for non-integers.
- Constrained the `CLZ` combiner to T32 because it is not clear whether it is profitable in A32 (where conditional moves are more accessible)
- Added negative tests to make sure no branches are emitted where we don't want them.


https://reviews.llvm.org/D34515

Files:
  lib/Target/ARM/ARMISelLowering.cpp
  test/CodeGen/ARM/atomic-cmpxchg.ll
  test/CodeGen/ARM/cmn.ll
  test/CodeGen/ARM/cmpxchg-O0.ll
  test/CodeGen/ARM/select-imm.ll
  test/CodeGen/Thumb/branchless-cmp.ll
  test/CodeGen/Thumb/constants.ll
  test/CodeGen/Thumb/long-setcc.ll
  test/CodeGen/Thumb2/float-cmp.ll
  test/CodeGen/Thumb2/thumb2-cmn.ll
  test/CodeGen/Thumb2/thumb2-cmn2.ll
  test/CodeGen/Thumb2/thumb2-cmp.ll
  test/CodeGen/Thumb2/thumb2-cmp2.ll
  test/CodeGen/Thumb2/thumb2-teq.ll
  test/CodeGen/Thumb2/thumb2-teq2.ll
  test/CodeGen/Thumb2/thumb2-tst.ll
  test/CodeGen/Thumb2/thumb2-tst2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34515.117834.patch
Type: text/x-patch
Size: 40082 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171005/eab582b4/attachment.bin>