From: DeLesley Hutchins <delesley@google.com>
Date: Tue, 3 Sep 2013 22:35:53 +0000 (+0000)
Subject: Consumed analysis: update to comments in test cases.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5094920177bba444d8d67f672b428999770583b9;p=clang

Consumed analysis: update to comments in test cases.
Patch by chris.wailes@gmail.com.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@189866 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/test/SemaCXX/warn-consumed-parsing.cpp b/test/SemaCXX/warn-consumed-parsing.cpp
index 674197f6e0..4408dfc7d7 100644
--- a/test/SemaCXX/warn-consumed-parsing.cpp
+++ b/test/SemaCXX/warn-consumed-parsing.cpp
@@ -6,8 +6,11 @@
 #define RETURN_TYPESTATE(State)  __attribute__ ((return_typestate(State)))
 #define TESTS_UNCONSUMED         __attribute__ ((tests_unconsumed))
 
-// FIXME: This warning is not generated if it appears bellow the AttrTester0
-//        class declaration.  Why?
+// FIXME: This test is here because the warning is issued by the Consumed
+//        analysis, not SemaDeclAttr.  The analysis won't run after an error
+//        has been issued.  Once the attribute propagation for template
+//        instantiation has been fixed, this can be moved somewhere else and the
+//        definition can be removed.
 int returnTypestateForUnconsumable() RETURN_TYPESTATE(consumed); // expected-warning {{return state set for an unconsumable type 'int'}}
 int returnTypestateForUnconsumable() {
   return 0;