[PATCH] D64099: [InstCombine] pow(C,x) -> exp2(log2(C)*x)
Evandro Menezes via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 5 07:50:18 PDT 2019
evandro added a comment.
In D64099#1569035 <https://reviews.llvm.org/D64099#1569035>, @xbolva00 wrote:
> I tested the first patch and the second revision with change to exp2 locally with simple examples to check for correctness and performance - and yes, I didn’t check previous revision with folding so a mistake was made.
>
> I have no access to the SPEC benchmarks.
What correctness tests did you run? If necessary, I can run SPEC to validate this patch at least on x86-64.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64099/new/
https://reviews.llvm.org/D64099
More information about the llvm-commits
mailing list