[llvm] [ConstantFPRange] Implement `ConstantFPRange::makeAllowedFCmpRegion` (PR #110082)

Yingwei Zheng via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 30 05:56:49 PDT 2024


================
@@ -117,9 +221,10 @@ ConstantFPRange::makeSatisfyingFCmpRegion(FCmpInst::Predicate Pred,
   return getEmpty(Other.getSemantics());
 }
 
-ConstantFPRange ConstantFPRange::makeExactFCmpRegion(FCmpInst::Predicate Pred,
-                                                     const APFloat &Other) {
-  return makeAllowedFCmpRegion(Pred, ConstantFPRange(Other));
+std::optional<ConstantFPRange>
+ConstantFPRange::makeExactFCmpRegion(FCmpInst::Predicate Pred,
+                                     const APFloat &Other) {
+  return std::nullopt;
----------------
dtcxzyw wrote:

Yeah. I plan to address it in a subsequent patch. This change only contains a correctness fix.


https://github.com/llvm/llvm-project/pull/110082


More information about the llvm-commits mailing list