Hi, Attached patch is to fix PR14922. Currently when print an attribute the GNU syntax will always be used, even if the attribute has no GNU syntax.The fix is to pass the syntax flag when constructing the Attr node and take that into consideration when printing the attribute. The name of actual attribute gets printed is read from table gen definition file so there is still some limitations, for example, when an attribute has multiple spellings, the first spelling is used; and the namespace of the attribute (in case it's a C++11 attribute) is not printed. I test the patch locally in my project which has access to Clang AST but I am not sure how to write a stand alone test to test the attribute pretty print. After this patch gets in I'll send another patch which updates the SemaDeclAttr to pass the actual syntax flag from AttributeList to Attr.
Thanks! Michael
attr-print.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
