Ping.<br><br><div class="gmail_quote">On Mon, Feb 11, 2013 at 2:24 PM, Michael Han <span dir="ltr"><<a href="mailto:fragmentshaders@gmail.com" target="_blank">fragmentshaders@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi rsmith,<br>
<br>
This patch improves attribute printing by taking the source location of attributes into account when print them out. Previously, we hard code the print out locations for attributes and this has some issues especially for C++11 attributes for example:<br>

void foo [[noreturn]] (); would be printed out as<br>
void foo () [[noreturn]]; which has a different semantic with source.<br>
This patch uses the source locations of the attributes and VarDecl / FunctionDecl to print out attributes at expected locations as described in source files. It also fixes several FIXMEs in existing tests.<br>
<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D395" target="_blank">http://llvm-reviews.chandlerc.com/D395</a><br>
<br>
Files:<br>
  lib/AST/DeclPrinter.cpp<br>
  test/SemaCXX/cxx11-attr-print.cpp<br>
  test/SemaCXX/attr-print.cpp<br>
  test/Sema/attr-print.c<br>
</blockquote></div><br>