Thanks, attached the test patch. SemaDeclAttr is slightly modified to include the spelling index on creating an Attr for the attributes that covered in tests, and I'll update all attributes later as a separate patch.<br>
<br>One issue found by the tests is the argument list of align attribute:<br><br>def Aligned : InheritableAttr {<br>  let Spellings = [GNU<"aligned">, GNU<"align">, CXX11<"gnu", "aligned">];<br>
  let Subjects = [NonBitField, NormalVar, Tag];<br>  let Args = [AlignedArgument<"Alignment">, BoolArgument<"IsMSDeclSpec">];<br>}<br><br>Here the IsMSDeclSpec argument is not modeled from the source format of this attribute (e.g. __attribute__((aligned(...))) will always take a single argument instead of two), so my feeling is this is a hack and should be fixed and I prefer including the fix in a separate patch.<br>
<br>Michael<br><br><div class="gmail_quote">On Wed, Jan 16, 2013 at 7:12 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
test/Sema/ast-print.c, test/SemaCXX/ast-print.cpp, and<br>
test/SemaCXX/cxx11-ast-print.cpp look like good places to start.<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Jan 16, 2013 at 7:09 PM, Michael Han <<a href="mailto:fragmentshaders@gmail.com">fragmentshaders@gmail.com</a>> wrote:<br>
> Agree, any suggestions on where / how to add the tests? I don't find any<br>
> existing regression tests that cover attributes print.<br>
><br>
> Michael<br>
><br>


</div></div></blockquote></div>