[PATCH] D41342: [InstCombine] Missed optimization in math expression: simplify calls exp functions

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 30 05:55:14 PST 2019


lebedev.ri added inline comments.


================
Comment at: test/Transforms/InstCombine/fmul-exp2.ll:74
+; CHECK-NEXT:    [[TMP2:%.*]] = fadd reassoc double [[TMP1]], [[D:%.*]]
+; CHECK-NEXT:    [[TMP3:%.*]] = call reassoc double @llvm.exp2.f64(double [[TMP2]])
+; CHECK-NEXT:    ret double [[TMP3]]
----------------
Hm that is interesting, somehow i have never seen that before.
So the FMF can be set on such intrinsic calls too.
Here, we only need `reassoc` on the original `fmul`, not the `exp`?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D41342/new/

https://reviews.llvm.org/D41342





More information about the llvm-commits mailing list