[llvm] ValueTracking: Merge fcmpImpliesClass and fcmpToClassTest (PR #66522)

Serge Pavlov via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 27 07:00:49 PST 2023


================
@@ -214,8 +214,10 @@ std::pair<Value *, FPClassTest> fcmpToClassTest(CmpInst::Predicate Pred,
                                                 const APFloat *ConstRHS,
                                                 bool LookThroughSrc = true);
 
-/// Compute the possible floating-point classes that \p LHS could be based on an
-/// fcmp returning true. Returns { TestedValue, ClassesIfTrue, ClassesIfFalse }
+/// Compute the possible floating-point classes that \p LHS could be based on
+/// fcmp \Pred \p LHS, \p RHS.
+///
+/// Returns { TestedValue, ClassesIfTrue, ClassesIfFalse }
----------------
spavloff wrote:

Use `\retue
```suggestion
/// \returns { TestedValue, ClassesIfTrue, ClassesIfFalse }
```

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


More information about the llvm-commits mailing list