================
@@ -829,22 +829,22 @@ static bool flipDemorganSide(SmallVectorImpl<FixItHint>
&Fixes,
const ASTContext &Ctx, const Expr *E,
std::optional<BinaryOperatorKind> OuterBO);
-/// Inverts \p BinOp, Removing \p Parens if they exist and are safe to remove.
-/// returns \c true if there is any issue building the Fixes, \c false
-/// otherwise.
-static bool
-flipDemorganBinaryOperator(SmallVectorImpl<FixItHint> &Fixes,
- const ASTContext &Ctx, const BinaryOperator *BinOp,
- std::optional<BinaryOperatorKind> OuterBO,
- const ParenExpr *Parens = nullptr) {
- switch (BinOp->getOpcode()) {
+/// Inverts \p BO, \p LHS, and \p RHS, Removing \p Parens if they exist and are
+/// safe to remove. returns \c true if there is any issue building the Fixes,
\c
+/// false otherwise.
+static bool flipDemorganBinaryOperator(
+ SmallVectorImpl<FixItHint> &Fixes, const ASTContext &Ctx,
+ BinaryOperatorKind BO, const Expr *LHS, const Expr *RHS, SourceRange Range,
----------------
vbvictor wrote:
Could we give a more meaningful name for `Range`? From where to where this
range is
https://github.com/llvm/llvm-project/pull/164141
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
