This revision was automatically updated to reflect the committed changes. Closed by commit rL314124: [Sema] Null check in BuildDeclarationNameExpr (authored by kongyi).
Changed prior to commit: https://reviews.llvm.org/D38158?vs=116279&id=116571#toc Repository: rL LLVM https://reviews.llvm.org/D38158 Files: cfe/trunk/lib/Sema/SemaExpr.cpp cfe/trunk/test/SemaCXX/typo-correction-crash.cpp Index: cfe/trunk/test/SemaCXX/typo-correction-crash.cpp =================================================================== --- cfe/trunk/test/SemaCXX/typo-correction-crash.cpp +++ cfe/trunk/test/SemaCXX/typo-correction-crash.cpp @@ -1,4 +1,4 @@ -// RUN: %clang_cc1 -fsyntax-only -std=c++14 -verify %s +// RUN: %clang_cc1 -fsyntax-only -std=c++17 -verify %s auto check1() { return 1; return s; // expected-error {{use of undeclared identifier 's'}} @@ -19,3 +19,5 @@ FooRecord::NestedNamespace::type x; // expected-error {{no member named 'NestedNamespace' in 'FooRecord'; did you mean 'BarNamespace::NestedNamespace'?}} void cast_expr(int g) { +int(n)(g); } // expected-error {{undeclared identifier 'n'}} + +void bind() { for (const auto& [test,_] : _test_) { }; } // expected-error {{undeclared identifier '_test_'}} Index: cfe/trunk/lib/Sema/SemaExpr.cpp =================================================================== --- cfe/trunk/lib/Sema/SemaExpr.cpp +++ cfe/trunk/lib/Sema/SemaExpr.cpp @@ -2803,6 +2803,8 @@ { QualType type = VD->getType(); + if (type.isNull()) + return ExprError(); if (auto *FPT = type->getAs<FunctionProtoType>()) { // C++ [except.spec]p17: // An exception-specification is considered to be needed when:
Index: cfe/trunk/test/SemaCXX/typo-correction-crash.cpp =================================================================== --- cfe/trunk/test/SemaCXX/typo-correction-crash.cpp +++ cfe/trunk/test/SemaCXX/typo-correction-crash.cpp @@ -1,4 +1,4 @@ -// RUN: %clang_cc1 -fsyntax-only -std=c++14 -verify %s +// RUN: %clang_cc1 -fsyntax-only -std=c++17 -verify %s auto check1() { return 1; return s; // expected-error {{use of undeclared identifier 's'}} @@ -19,3 +19,5 @@ FooRecord::NestedNamespace::type x; // expected-error {{no member named 'NestedNamespace' in 'FooRecord'; did you mean 'BarNamespace::NestedNamespace'?}} void cast_expr(int g) { +int(n)(g); } // expected-error {{undeclared identifier 'n'}} + +void bind() { for (const auto& [test,_] : _test_) { }; } // expected-error {{undeclared identifier '_test_'}} Index: cfe/trunk/lib/Sema/SemaExpr.cpp =================================================================== --- cfe/trunk/lib/Sema/SemaExpr.cpp +++ cfe/trunk/lib/Sema/SemaExpr.cpp @@ -2803,6 +2803,8 @@ { QualType type = VD->getType(); + if (type.isNull()) + return ExprError(); if (auto *FPT = type->getAs<FunctionProtoType>()) { // C++ [except.spec]p17: // An exception-specification is considered to be needed when:
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits