[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