[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