[PATCH] D32092: Attribute inline

Richard Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 14 12:55:22 PDT 2017


rsmith added inline comments.


================
Comment at: include/clang/Basic/Attr.td:869
 def GNUInline : InheritableAttr {
-  let Spellings = [GCC<"gnu_inline">];
+  let Spellings = [GCC<"gnu_inline">, Declspec<"inline">];
   let Subjects = SubjectList<[Function]>;
----------------
aaron.ballman wrote:
> I cannot see any documentation on MSDN for this spelling of the attribute, and MSVC 2015 rejects it. What is the motivating use case for this spelling?
Also, it seems rather unlikely that MSVC would implement the GNU inline semantics. Have you investigated the actual semantic effects of this `__declspec` attribute and checked they match the GNU semantics?


https://reviews.llvm.org/D32092





More information about the cfe-commits mailing list