rsmith added inline comments.

================
Comment at: include/clang/Basic/Attr.td:201
@@ -200,2 +200,3 @@
 class Declspec<string name> : Spelling<name, "Declspec">;
+class Microsoft<string name> : Spelling<name, "Microsoft">;
 class CXX11<string namespace, string name, int version = 1>
----------------
Given that MS have deprecated this syntax and are trying to move away from it 
towards `__declspec`, I think calling this "Microsoft" is a mistake. As MS' 
documentation calls these "ATL attributes", it would seem reasonable for us to 
do the same.


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