[Parser] Correct initalizer typos before lambda capture type is deduced.
This is the same assertion as in https://reviews.llvm.org/D25206 that is triggered when RecordLayoutBuilder tries to compute the size of a field (for capture "typo_boo" in the test case) whose type hasn't been deduced. The fix is to add CorrectDelayedTyposInExpr call to the cases when we aren't disambiguating between an Obj-C message send and a lambda expression. rdar://problem/31760839 Reviewers: rsmith, ahatanak Reviewed By: arphaman Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D36853 llvm-svn: 311480
This commit is contained in:
parent
6a71f364f1
commit
b0f1aae9b3
|
@ -966,6 +966,8 @@ Optional<unsigned> Parser::ParseLambdaIntroducer(LambdaIntroducer &Intro,
|
||||||
// that would be an error.
|
// that would be an error.
|
||||||
|
|
||||||
ParsedType InitCaptureType;
|
ParsedType InitCaptureType;
|
||||||
|
if (!Init.isInvalid())
|
||||||
|
Init = Actions.CorrectDelayedTyposInExpr(Init.get());
|
||||||
if (Init.isUsable()) {
|
if (Init.isUsable()) {
|
||||||
// Get the pointer and store it in an lvalue, so we can use it as an
|
// Get the pointer and store it in an lvalue, so we can use it as an
|
||||||
// out argument.
|
// out argument.
|
||||||
|
|
|
@ -206,3 +206,11 @@ void test(double weight) {
|
||||||
find(weight); // expected-note {{in instantiation of function template specialization}}
|
find(weight); // expected-note {{in instantiation of function template specialization}}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
namespace init_capture_undeclared_identifier {
|
||||||
|
auto a = [x = y]{}; // expected-error{{use of undeclared identifier 'y'}}
|
||||||
|
|
||||||
|
int typo_foo; // expected-note 2 {{'typo_foo' declared here}}
|
||||||
|
auto b = [x = typo_boo]{}; // expected-error{{use of undeclared identifier 'typo_boo'; did you mean 'typo_foo'}}
|
||||||
|
auto c = [x(typo_boo)]{}; // expected-error{{use of undeclared identifier 'typo_boo'; did you mean 'typo_foo'}}
|
||||||
|
}
|
||||||
|
|
Loading…
Reference in New Issue