[PATCH] fold: sqrt(x * x * y) -> fabs(x) * sqrt(y)

hfinkel at anl.gov hfinkel at anl.gov
Wed Oct 15 18:02:58 PDT 2014


Adding for the record...

================
Comment at: lib/Transforms/InstCombine/InstCombineMulDivRem.cpp:453
@@ +452,3 @@
+  // A function-level attribute overrides the instruction-level fast-math-flags.
+  // This can happen when code is inlined via LTO.
+  if (!AllowReassociate) {
----------------
As we discussed elsewhere (PR21291), this is incorrect.

http://reviews.llvm.org/D5787






More information about the llvm-commits mailing list