[PATCH] D37019: Add select simplifications

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 26 15:31:27 PDT 2017


mcrosier added a comment.

In https://reviews.llvm.org/D37019#881221, @mcrosier wrote:

> Unfortunately, we're seeing a 4.7% regression in SPEC2006/lbm due to this change.  The number of dynamic instructions is increased by ~5.5%.
>
> We're running on AArch64/Falkor with the following flags: -O3 -fno-math-errno -ffp-contract=fast -fomit-frame-pointer
>
> All of the code changes are in LBM_performStreamCollide(), which accounts for ~98% of execution.
>
> I'll keep digging so I can provide more details, but I wanted to report the regression.


Balaram posted a candidate fix here: https://reviews.llvm.org/D38288


Repository:
  rL LLVM

https://reviews.llvm.org/D37019





More information about the llvm-commits mailing list