thakis 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> ---------------- rsmith wrote: > 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. As said upthread, I'm happy to rename this to whatever you all agree on (or what you decree given that you're code owner), but they are called Microsoft attributes in today's code already, and renaming that is unrelated to this CL. If you think it's important, I'm happy to rename the existing code before lading this CL and then using the new name in this CL immediately.
Re "ATL attributes": I think some of the attributes in [] are SAL attributes which as far as I know is independent of ATL. Are you sure the documentation refers to the [] syntax, or could it refer to specific attributes in that syntax? https://reviews.llvm.org/D23895 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
