[PATCH] D35821: Have ARM call setBooleanContents(ZeroOrOneBooleanContent).

Joel Galenson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 09:40:18 PDT 2017


jgalenson updated this revision to Diff 108102.
jgalenson added a comment.

Adding an i1 parameter saves two of the testcases.  Thanks for the idea.

This does make the code worse on these benchmarks, mainly by doing things like moving 0 into an undef that is used as a condition.  However, I compiled a dozen or so of the C programs in the LLVM test suite and there was no difference in the generated code.


https://reviews.llvm.org/D35821

Files:
  lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
  lib/Target/ARM/ARMISelLowering.cpp
  test/CodeGen/ARM/cse-call.ll
  test/CodeGen/ARM/i1.ll
  test/CodeGen/ARM/sub-cmp-peephole.ll
  test/CodeGen/Thumb2/2009-09-28-ITBlockBug.ll
  test/CodeGen/Thumb2/2010-11-22-EpilogueBug.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35821.108102.patch
Type: text/x-patch
Size: 5885 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170725/3ce49721/attachment.bin>


More information about the llvm-commits mailing list