[PATCH] Improve attributes printing.

Michael Han fragmentshaders at gmail.com
Tue Feb 19 07:44:27 PST 2013


Ping.

On Mon, Feb 11, 2013 at 2:24 PM, Michael Han <fragmentshaders at gmail.com>wrote:

> Hi rsmith,
>
> 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:
> void foo [[noreturn]] (); would be printed out as
> void foo () [[noreturn]]; which has a different semantic with source.
> 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.
>
>
>
> http://llvm-reviews.chandlerc.com/D395
>
> Files:
>   lib/AST/DeclPrinter.cpp
>   test/SemaCXX/cxx11-attr-print.cpp
>   test/SemaCXX/attr-print.cpp
>   test/Sema/attr-print.c
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130219/008ee157/attachment.html>


More information about the cfe-commits mailing list