From: Ted Kremenek <kremenek@apple.com>
Date: Sat, 28 Aug 2010 00:19:12 +0000 (+0000)
Subject: Update test case, with comment to later investigate the correct behavior.  Now the... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2c1b9c3fa3d920aa6ccf843b40c1eb0f1ad04d58;p=clang

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
---

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;