Skip to content

Commit 4382997

Browse files
committed
[ValueTracking] remove stale comments; NFC
The checks were improved with: https://reviews.llvm.org/rL290194 llvm-svn: 290826
1 parent 67ada75 commit 4382997

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

llvm/lib/Analysis/ValueTracking.cpp

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2542,9 +2542,6 @@ bool llvm::CannotBeNegativeZero(const Value *V, const TargetLibraryInfo *TLI,
25422542
if (const ConstantFP *CFP = dyn_cast<ConstantFP>(V))
25432543
return !CFP->getValueAPF().isNegZero();
25442544

2545-
// FIXME: Magic number! At the least, this should be given a name because it's
2546-
// used similarly in CannotBeOrderedLessThanZero(). A better fix may be to
2547-
// expose it as a parameter, so it can be used for testing / experimenting.
25482545
if (Depth == MaxDepth)
25492546
return false; // Limit search depth.
25502547

@@ -2589,9 +2586,6 @@ bool llvm::CannotBeOrderedLessThanZero(const Value *V,
25892586
if (const ConstantFP *CFP = dyn_cast<ConstantFP>(V))
25902587
return !CFP->getValueAPF().isNegative() || CFP->getValueAPF().isZero();
25912588

2592-
// FIXME: Magic number! At the least, this should be given a name because it's
2593-
// used similarly in CannotBeNegativeZero(). A better fix may be to
2594-
// expose it as a parameter, so it can be used for testing / experimenting.
25952589
if (Depth == MaxDepth)
25962590
return false; // Limit search depth.
25972591

0 commit comments

Comments
 (0)