diff options
| -rw-r--r-- | clang/lib/Sema/SemaOverload.cpp | 3 | ||||
| -rw-r--r-- | clang/test/SemaCXX/operator-arrow-temporary.cpp | 19 |
2 files changed, 21 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaOverload.cpp b/clang/lib/Sema/SemaOverload.cpp index c023cd3b2a4..5b444c14d6f 100644 --- a/clang/lib/Sema/SemaOverload.cpp +++ b/clang/lib/Sema/SemaOverload.cpp @@ -8934,7 +8934,8 @@ Sema::BuildOverloadedArrowExpr(Scope *S, Expr *Base, SourceLocation OpLoc) { if (CheckCallReturnType(Method->getResultType(), OpLoc, TheCall, Method)) return ExprError(); - return Owned(TheCall); + + return MaybeBindToTemporary(TheCall); } /// FixOverloadedFunctionReference - E is an expression that refers to diff --git a/clang/test/SemaCXX/operator-arrow-temporary.cpp b/clang/test/SemaCXX/operator-arrow-temporary.cpp new file mode 100644 index 00000000000..8e79609342b --- /dev/null +++ b/clang/test/SemaCXX/operator-arrow-temporary.cpp @@ -0,0 +1,19 @@ +// RUN: %clang_cc1 -fsyntax-only -verify %s +// PR9615 + +struct Resource { + void doit(); +}; + +template<int x> struct Lock { + ~Lock() { int a[x]; } // expected-error {{declared as an array with a negative size}} + Resource* operator->() { return 0; } +}; + +struct Accessor { + Lock<-1> operator->(); +}; + +// Make sure we try to instantiate the destructor for Lock here +void f() { Accessor acc; acc->doit(); } // expected-note {{requested here}} + |
