[cfe-dev] -Wambiguous-reversed-operator with equal types

Nuno Lopes via cfe-dev cfe-dev at lists.llvm.org
Thu Dec 10 06:46:29 PST 2020


I've been trying to fix a bunch of LLVM headers so they are compatible with
C++20. (so I can include those from Alive2 source code)

I'm currently stuck with -Wambiguous-reversed-operator. I've fixed a couple,
but this one (a few similar) left me wondering if the warning is correct or

In file included from llvm/include/llvm/Passes/PassBuilder.h:19:
In file included from llvm/include/llvm/Analysis/CGSCCPassManager.h:98:
llvm/include/llvm/Analysis/LazyCallGraph.h:1117:22: error: ISO C++20
considers use of overloaded operator '!=' (with operand types
'llvm::User::value_op_iterator' and 'llvm::User::value_op_iterator') to be
ambiguous despite there being a unique best viable function
      for (Value *Op : C->operand_values())
llvm/include/llvm/ADT/iterator.h:263:8: note: ambiguity is between a regular
call to this operator and a call with the argument order reversed
  bool operator==(const DerivedT &RHS) const { return I == RHS.I; }

Note that it's complaining about comparing 'llvm::User::value_op_iterator'
and 'llvm::User::value_op_iterator' (i.e., same type). Since both arguments
are of the same type and the comparison function has const on both LHS/RHS,
reversing them shouldn't have any impact, right?
Unless there's some type conversion going on that the error message is

Could someone please confirm if the warning is correct and/or the code needs


More information about the cfe-dev mailing list