[PATCH] D105996: [AArch64] Enable Upper bound unrolling universally

JinGu Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 29 09:01:10 PDT 2021


jaykang10 added a comment.

In D105996#2913839 <https://reviews.llvm.org/D105996#2913839>, @dmgreen wrote:

> Do you have number for other benchmarks too? I believe that whether the option is beneficial - although it can depend on the sophistication of the cpu's branch predictor - is more dependent on the code that is being compiled. Some code is better suited for this type of unrolling, in cases because it helps further simplification or it just happens not to run into the drawbacks of the extra branches. It appears that SPEC is a win - I'm wondering if the llvm-test-suite would be the same as a larger selection of tests.

Yep, let me try to get the number from llvm-test-suite too.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105996/new/

https://reviews.llvm.org/D105996



More information about the llvm-commits mailing list