diff --git a/clang/lib/Sema/SemaExprCXX.cpp b/clang/lib/Sema/SemaExprCXX.cpp index 27ec863e7a35e9..6c2c61639be773 100644 --- a/clang/lib/Sema/SemaExprCXX.cpp +++ b/clang/lib/Sema/SemaExprCXX.cpp @@ -843,10 +843,10 @@ Sema::ActOnCXXThrow(Scope *S, SourceLocation OpLoc, Expr *Ex) { break; } + // FIXME: Many of the scope checks here seem incorrect. if (S->getFlags() & (Scope::FnScope | Scope::ClassScope | Scope::BlockScope | - Scope::FunctionPrototypeScope | Scope::ObjCMethodScope | - Scope::TryScope)) + Scope::ObjCMethodScope | Scope::TryScope)) break; } } diff --git a/clang/test/CXX/class/class.init/class.copy.elision/p3.cpp b/clang/test/CXX/class/class.init/class.copy.elision/p3.cpp index d7b52c561a5478..975557c5187bbe 100644 --- a/clang/test/CXX/class/class.init/class.copy.elision/p3.cpp +++ b/clang/test/CXX/class/class.init/class.copy.elision/p3.cpp @@ -98,7 +98,7 @@ struct A1 { A1(const A1 &); A1(A1 &&) = delete; // expected-note@-1 2{{'A1' has been explicitly marked deleted here}} - // cxx11_2b-note@-2 {{'A1' has been explicitly marked deleted here}} + // cxx11_2b-note@-2 3{{'A1' has been explicitly marked deleted here}} }; void test1() { try { @@ -132,10 +132,10 @@ void test3(A1 a) try { namespace PR54341 { void test4(A1 a) { void f(decltype((throw a, 0))); - // expected-warning@-1 {{has no effect}} + // expected-error@-1 {{call to deleted constructor of 'test_throw_parameter::A1'}} void g(int = decltype(throw a, 0){}); - // expected-warning@-1 {{has no effect}} + // expected-error@-1 {{call to deleted constructor of 'test_throw_parameter::A1'}} } void test5(A1 a, int = decltype(throw a, 0){}) {}