[PATCH] D50900: [SimplifyLibCalls] Support pow(x, -0.5) in replacePowWithSqrt without NSZ/noInfs.

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 17 13:05:13 PDT 2018


evandro added a comment.

Can you please rebase it, for https://reviews.llvm.org/D50036 touched the same function?



================
Comment at: lib/Transforms/Utils/SimplifyLibCalls.cpp:1228
+    if (ExpoF->isNegative()) {
+      Value *PositiveZero = ConstantFP::get(Ty, 0.0);
+      Value *FCmp = B.CreateFCmpOEQ(Base, NegInf);
----------------
This one and the line below could be shared with the `else` clause.


https://reviews.llvm.org/D50900





More information about the llvm-commits mailing list