[PATCH] D53942: IR Outliner Pass

[Yvan Roux via Phabricator via llvm-commits]

yroux added a comment.

Hi Tom,

I've tested your patch on 32 bits ARM target (in thumb2 mode) on top of my
Machine Outliner prototype.  The code size reduction is not as good as good as
the ones reported a year ago on x86 and AArch64 (around 1% in geomean over Oz
for Spec2k6 and 1.7% when combined with the Machine Outliner), but it is the
same thing for the Machine Outliner in Thumb2, and it is still interesting, so
thanks for the work done.

On CoremarkPro, which is not good candidate for Ouliners, it performs a bit
better than the Machine Ouliner, but there is a build issue for sha256.c when
early and late IR Outlining are combined.  I didn't had time to make a reduce
test case, but let me know if you are not able to reproduce it.


Repository:
  rL LLVM

https://reviews.llvm.org/D53942