maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Pass the right SourceLocation to Actions.ActOnOverloadedOperatorReferenceExpr and Actions.ActOnConversionOperatorReferenceExpr. Update incomplete-call.cpp test. 

llvm-svn: 84026
This commit is contained in:
Anders Carlsson 2009-10-13 21:02:07 +00:00
parent 39a029d8a3
commit 8523d20af3
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
clang/lib/Parse/ParseExpr.cpp
View File

@ -983,12 +983,14 @@ Parser::ParsePostfixExpressionSuffix(OwningExprResult LHS) {
} else if (getLang().CPlusPlus && Tok.is(tok::kw_operator)) { } else if (getLang().CPlusPlus && Tok.is(tok::kw_operator)) {
// We have a reference to a member operator, e.g., t.operator int or // We have a reference to a member operator, e.g., t.operator int or
// t.operator+. // t.operator+.
SourceLocation OperatorLoc = Tok.getLocation();
if (OverloadedOperatorKind Op = TryParseOperatorFunctionId()) { if (OverloadedOperatorKind Op = TryParseOperatorFunctionId()) {
if (!LHS.isInvalid()) if (!LHS.isInvalid())
LHS = Actions.ActOnOverloadedOperatorReferenceExpr(CurScope, LHS = Actions.ActOnOverloadedOperatorReferenceExpr(CurScope,
move(LHS), OpLoc, move(LHS), OpLoc,
OpKind, OpKind,
Tok.getLocation(), OperatorLoc,
Op, &SS); Op, &SS);
// TryParseOperatorFunctionId already consumed our token, so // TryParseOperatorFunctionId already consumed our token, so
// don't bother // don't bother
@ -997,7 +999,7 @@ Parser::ParsePostfixExpressionSuffix(OwningExprResult LHS) {
LHS = Actions.ActOnConversionOperatorReferenceExpr(CurScope, LHS = Actions.ActOnConversionOperatorReferenceExpr(CurScope,
move(LHS), OpLoc, move(LHS), OpLoc,
OpKind, OpKind,
Tok.getLocation(), OperatorLoc,
ConvType, &SS); ConvType, &SS);
} else { } else {
// Don't emit a diagnostic; ParseConversionFunctionId does it for us // Don't emit a diagnostic; ParseConversionFunctionId does it for us

7
clang/test/SemaCXX/incomplete-call.cpp
View File

@ -1,10 +1,12 @@
// RUN: clang-cc -fsyntax-only -verify %s // RUN: clang-cc -fsyntax-only -verify %s
struct A; // expected-note 4 {{forward declaration of 'struct A'}} struct A; // expected-note 6 {{forward declaration of 'struct A'}}
A f(); // expected-note {{note: 'f' declared here}} A f(); // expected-note {{note: 'f' declared here}}
struct B { struct B {
A f(); // expected-note {{'f' declared here}} A f(); // expected-note {{'f' declared here}}
A operator()(); // expected-note {{'operator()' declared here}}
operator A(); // expected-note {{'operator A' declared here}}
}; };
void g() { void g() {
@ -17,4 +19,7 @@ void g() {
B b; B b;
b.f(); // expected-error {{calling 'f' with incomplete return type 'struct A'}} b.f(); // expected-error {{calling 'f' with incomplete return type 'struct A'}}
b.operator()(); // expected-error {{calling 'operator()' with incomplete return type 'struct A'}}
b.operator A(); // expected-error {{calling 'operator A' with incomplete return type 'struct A'}}
} }