aaron.ballman added inline comments.

================
Comment at: lib/Parse/ParseDeclCXX.cpp:4031
@@ +4030,3 @@
+      ConsumeToken();
+      if (Name->getName() == "uuid" && Tok.is(tok::l_paren))
+        ParseMicrosoftUuidAttributeArgs(Name, Loc, attrs);
----------------
Silently ignoring seems like the wrong thing to do -- can we diagnose (even if 
it's a less-than-ideal diagnostics with a fixme)?

================
Comment at: test/Parser/ms-square-bracket-attributes.mm:13
@@ +12,3 @@
+[uuid( "000000A0-0000-0000-C000-000000000049" )] struct struct_with_uuid;
+
+// Optionally, the uuid can be surrounded by one set of braces.
----------------
Diagnosing the second case in a follow-up patch is totally fine by me.


https://reviews.llvm.org/D23895



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to