[clang] [Clang][Sema] Use the correct lookup context when building overloaded 'operator->' in the current instantiation (PR #104458)

via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 3 09:01:48 PDT 2024


================
@@ -7939,7 +7927,14 @@ ExprResult Sema::ActOnStartCXXMemberReference(Scope *S, Expr *Base,
           }
         }
         return ExprError();
+      } else if (Result.isUnset()) {
+        // BuildOverloadedArrowExpr returns an empty expression to indicate
+        // that we need to build a dependent overloaded arrow expression.
----------------
cor3ntin wrote:

I'm not sure RebuildCXXOperatorCallExpr and its caller would deal with an ExprEmpty when calling  
ActOnOverloadedArrowExpr

So yeah, maybe output parameter + assert in treetransform would be better?

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


More information about the cfe-commits mailing list