[PATCH] D43160: [InstSimplify] allow exp/log simplifications with only 'reassoc' FMF

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: test/Transforms/InstSimplify/exp-intrinsic.ll:22
 ;
   %1 = call fast double @llvm.log.f64(double %a)
+  %2 = call reassoc double @llvm.exp.f64(double %1)
----------------
wristow wrote:
> 'reassoc' instead of 'fast' should also be all that's needed for the llvm.log.f64 call to enable this transformation.
Hmm...looking a bit closer, this test doesn't add much value. We show that the FMF on the first instruction is irrelevant in the next test (and the same is true in each file). Should I just delete this and its siblings?


https://reviews.llvm.org/D43160