[PATCH] D14357: [ARM] Enable shrink-wrapping by default

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 4 16:20:48 PST 2015


qcolombet created this revision.
qcolombet added reviewers: t.p.northover, rengolin.
qcolombet added a subscriber: llvm-commits.
qcolombet set the repository for this revision to rL LLVM.
Herald added subscribers: rengolin, aemerson.

Hi,

This patch enables shrink-wrapping by default for ARM.
I have tested shrink-wrapping and no correctness problems came out or runtime regressions.
Although the gain may be small, the code looks nicer with shrink-wrapping enabled and the performance could just be better.

For the record shrink-wrapping was introduced after this review:
http://reviews.llvm.org/D9210

More information on Shrink-Wrapping:
http://lists.llvm.org/pipermail/llvm-dev/2015-May/085220.html

OK to commit?

Thanks,
-Quentin

Repository:
  rL LLVM

http://reviews.llvm.org/D14357

Files:
  lib/Target/ARM/ARMFrameLowering.h
  test/CodeGen/ARM/call-tc.ll
  test/CodeGen/ARM/fold-stack-adjust.ll
  test/CodeGen/ARM/ifcvt5.ll
  test/CodeGen/ARM/ifcvt6.ll
  test/CodeGen/ARM/ifcvt8.ll
  test/CodeGen/ARM/machine-cse-cmp.ll
  test/CodeGen/ARM/thumb-alignment.ll
  test/CodeGen/Thumb2/ifcvt-compare.ll
  test/CodeGen/Thumb2/thumb2-ifcvt1.ll
  test/CodeGen/Thumb2/thumb2-ifcvt2.ll
  test/CodeGen/Thumb2/v8_IT_5.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14357.39285.patch
Type: text/x-patch
Size: 6182 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151105/a0b95402/attachment.bin>


More information about the llvm-commits mailing list