[PATCH] D41381: [InstCombine] Missed optimization in math expression: squashing exp(log), log(exp)
Davide Italiano via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 20 01:32:13 PST 2017
davide requested changes to this revision.
davide added a subscriber: scanon.
davide added a comment.
This revision now requires changes to proceed.
I don't have access to my checkout here, but I'm fairly sure we do something similar in SimplifyLibCalls.
I understand many of the pattern proposed will be duplicated, which is, not ideal.
Also, I'm pretty sure @scanon once taught me (when I implemented a similar transformation in simplifylibcalls) that these can underflow/overflow pretty dramatically, so, shouldn't these only be enabled under `-ffast-math` ?
https://reviews.llvm.org/D41381
More information about the llvm-commits
mailing list