[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