[PATCH] D23052: [Inliner] Add a flag to disable manual alloca merging in the Inliner.

Balaram Makam via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 08:12:07 PDT 2016


bmakam added a comment.

In https://reviews.llvm.org/D23052#508976, @mcrosier wrote:

> @bmakam: Would you mind downloading and testing this patch?  Please do full correctness and SPEC200X performance (our head-to-head methodology with train input would be fine).


Tested on Kryo. Only these SPEC200X benchmarks had non-noise performance gains(runtime) with this change:

| spec2006/soplex:train    | [-2.347%, +3.157%] |
| spec2006/sphinx3:train   | [+0.469%, +2.124%] |
| spec2006/xalancbmk:train | [+6.566%, +8.743%] |

There were no regressions.


https://reviews.llvm.org/D23052





More information about the llvm-commits mailing list