[LVI] Remove unused get of Range metadata.

[andjo403]

There is no test for this code and it also do not change the result of https://github.com/dtcxzyw/llvm-opt-benchmark when I run it locally.
I have not been able to make a test for this and do not know how to handle the range attribute for this so maybe it shall be removed?

CC @nikic

[llvmbot]

@llvm/pr-subscribers-llvm-analysis

Author: Andreas Jonson (andjo403)

Changes

There is no test for this code and it also do not change the result of https://github.com/dtcxzyw/llvm-opt-benchmark when I run it locally.
I have not been able to make a test for this and do not know how to handle the range attribute for this so maybe it shall be removed?

CC @nikic

---

Full diff: https://github.com/llvm/llvm-project/pull/94914.diff

1 Files Affected:

• (modified) llvm/lib/Analysis/LazyValueInfo.cpp (-3)

diff --git a/llvm/lib/Analysis/LazyValueInfo.cpp b/llvm/lib/Analysis/LazyValueInfo.cpp
index 6cded828c25f4..8b6e56ce8113d 100644
--- a/llvm/lib/Analysis/LazyValueInfo.cpp
+++ b/llvm/lib/Analysis/LazyValueInfo.cpp
@@ -1116,9 +1116,6 @@ LazyValueInfoImpl::getValueFromSimpleICmpCondition(CmpInst::Predicate Pred,
     if (!R)
       return std::nullopt;
     RHSRange = toConstantRange(*R, RHS->getType());
-  } else if (Instruction *I = dyn_cast<Instruction>(RHS)) {
-    if (auto *Ranges = I->getMetadata(LLVMContext::MD_range))
-      RHSRange = getConstantRangeFromMetadata(*Ranges);
   }
 
   ConstantRange TrueValues =

[nikic]

LGTM. This code predates the ability to use getBlockValue() when reasoning about conditions. It could theoretically still be useful for the cases that currently pass UseBlockValue=false, but these are also less important, and this is a pretty weird special case.