[PATCH] D28405: SimplifyLibCalls: Replace fabs libcalls with intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:1646
+      NewFabs->takeName(II);
+      return new FPExtInst(NewFabs, II->getType());
+    }
----------------
efriedma wrote:
> Do we need to preserve fast-math flags here?
I don't think fpext supports fast math flags


https://reviews.llvm.org/D28405