[PATCH] D41381: [InstCombine] Missed optimization in math expression: squashing exp(log), log(exp)

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

Since this hasn't been committed yet, let me ask: why are we doing this in instcombine rather than instsimplify? When no new instructions are created, we generally want that transform in InstSimplify because that makes passes like GVN more powerful. InstSimplify already has "SimplifyIntrinsic", so I think this could fit neatly in there.


https://reviews.llvm.org/D41381