From 2c1b9c3fa3d920aa6ccf843b40c1eb0f1ad04d58 Mon Sep 17 00:00:00 2001 From: Ted Kremenek Date: Sat, 28 Aug 2010 00:19:12 +0000 Subject: [PATCH] Update test case, with comment to later investigate the correct behavior. Now the behavior is at least consistent. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@112335 91177308-0d34-0410-b5e6-96231b3b80d8 --- test/SemaCXX/return-noreturn.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/SemaCXX/return-noreturn.cpp b/test/SemaCXX/return-noreturn.cpp index 9242d1240a..dfd5487321 100644 --- a/test/SemaCXX/return-noreturn.cpp +++ b/test/SemaCXX/return-noreturn.cpp @@ -7,9 +7,12 @@ namespace PR6884 { ~abort_struct() __attribute__((noreturn)); }; + // FIXME: Should either of these actually warn, since the destructor is + // marked noreturn? + int f() { abort_struct(); - } + } // expected-warning{{control reaches end of non-void function}} int f2() { abort_struct s; -- 2.50.1