[PATCH] D24414: Conditionally eliminate library calls where the result value is not used

David Li via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 17 09:55:40 PDT 2016

davidxl accepted this revision.
davidxl added a comment.
This revision is now accepted and ready to land.


(the original patch also includes handling of math libs which are not properly annotated by Clang FE. Is there a patch submitted for that fix?)

Comment at: lib/Transforms/Utils/LibCallsShrinkWrap.cpp:320
+  Type *ArgType = CI.getArgOperand(0)->getType();
+  if (!(ArgType->isFloatTy() || ArgType->isDoubleTy() ||
Add a comment (todo) for handle long double in other formats (either with conservative range test or precise handling).


More information about the llvm-commits mailing list