[PATCH] D28484: SimplifyLibCalls: Replace more unary libcalls with intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Transforms/Utils/SimplifyLibCalls.cpp:2127-2137
+      return replaceUnaryCall(CI, Builder, Intrinsic::ceil);
     case LibFunc::floor:
+      return replaceUnaryCall(CI, Builder, Intrinsic::floor);
     case LibFunc::rint:
+      return replaceUnaryCall(CI, Builder, Intrinsic::rint);
     case LibFunc::round:
+      return replaceUnaryCall(CI, Builder, Intrinsic::round);
----------------
davide wrote:
> Does your code handle double-float shrinking correctly? (just checking)
The InstCombine part of the patch adds the shrinking with the intrinsics


https://reviews.llvm.org/D28484