[libcxx-commits] [lldb] [clang] [llvm] [flang] [libc] [compiler-rt] [libcxx] [clang] Fix assertion failure with deleted overloaded unary operators (PR #78316)

Erich Keane via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jan 17 06:15:36 PST 2024


================
@@ -14310,8 +14310,8 @@ Sema::CreateOverloadedUnaryOp(SourceLocation OpLoc, UnaryOperatorKind Opc,
         PartialDiagnosticAt(OpLoc, PDiag(diag::err_ovl_deleted_oper)
                                        << UnaryOperator::getOpcodeStr(Opc)
                                        << Input->getSourceRange()),
-        *this, OCD_AllCandidates, ArgsArray, UnaryOperator::getOpcodeStr(Opc),
-        OpLoc);
+        *this, OCD_AllCandidates, ArgsArray.slice(1),
----------------
erichkeane wrote:

IMO, `ArgsArray.drop_front()` makes more sense to me here.  It does the same thing, but is, IMO, more 'clear'.

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


More information about the libcxx-commits mailing list