[PATCH] D27932: InstSimplify: Eliminate fabs on known positive

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 11:29:48 PST 2017


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

LGTM.



================
Comment at: include/llvm/Analysis/ValueTracking.h:179
+  bool SignBitMustBeZero(const Value *V, const TargetLibraryInfo *TLI,
+                         unsigned Depth = 0);
+
----------------
Minor tweak: Depth is never explicitly passed in, so might as well get rid of the argument.


https://reviews.llvm.org/D27932





More information about the llvm-commits mailing list