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

[Joel Galenson via Phabricator via llvm-commits]

jgalenson updated this revision to Diff 111022.
jgalenson edited the summary of this revision.
jgalenson added a comment.

Thanks for the suggestions, Renato.  This now includes only the setBooleanContents part of the patch.  Removing the SelectionDAG optimization causes wide-compares.ll to generate worse code, but no other test is affected.  If people would like, I can easily submit a follow-up patch with that change.

As for the i1 test, I couldn't figure out how to get it to work with a cast, so I instead I removed some of the overly-specific CHECK lines.  It now simply checks for the "mov r0, #0" before the cmp.


https://reviews.llvm.org/D35821

Files:
  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/ARM/wide-compares.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.111022.patch
Type: text/x-patch
Size: 12087 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170814/6924b519/attachment.bin>