[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