r197582 - Adding some comments about AST node requirements for attributes which create AST nodes but never actually make use of them. No functional changes.

Aaron Ballman aaron at aaronballman.com
Wed Dec 18 07:59:41 PST 2013


Author: aaronballman
Date: Wed Dec 18 09:59:41 2013
New Revision: 197582

URL: http://llvm.org/viewvc/llvm-project?rev=197582&view=rev
Log:
Adding some comments about AST node requirements for attributes which create AST nodes but never actually make use of them. No functional changes.

Modified:
    cfe/trunk/include/clang/Basic/Attr.td

Modified: cfe/trunk/include/clang/Basic/Attr.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/Attr.td?rev=197582&r1=197581&r2=197582&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/Attr.td (original)
+++ cfe/trunk/include/clang/Basic/Attr.td Wed Dec 18 09:59:41 2013
@@ -502,12 +502,17 @@ def GNUInline : InheritableAttr {
 def Hot : InheritableAttr {
   let Spellings = [GNU<"hot">, CXX11<"gnu", "hot">];
   let Subjects = SubjectList<[Function]>;
+  // An AST node is created for this attribute, but not actually used beyond
+  // semantic checking for mutual exclusion with the Cold attribute.
 }
 
 def IBAction : InheritableAttr {
   let Spellings = [GNU<"ibaction">];
   let Subjects = SubjectList<[ObjCInstanceMethod], WarnDiag,
                              "ExpectedObjCInstanceMethod">;
+  // An AST node is created for this attribute, but is not used by other parts
+  // of the compiler. However, this node needs to exist in the AST because
+  // external tools rely on it.
 }
 
 def IBOutlet : InheritableAttr {





More information about the cfe-commits mailing list