<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, 30 Aug 2018 at 12:27, Aaron Ballman via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Aug 30, 2018 at 3:21 PM, Richard Smith - zygoloid via<br>
Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br>
> rsmith marked an inline comment as done.<br>
> rsmith added inline comments.<br>
><br>
><br>
> ================<br>
> Comment at: utils/TableGen/ClangAttrEmitter.cpp:3881<br>
> +    SpellingKind K = (SpellingKind)Kind;<br>
> +    // FIXME: Why are Microsoft spellings not listed?<br>
> +    if (K == SpellingKind::Microsoft)<br>
> ----------------<br>
> aaron.ballman wrote:<br>
>> We don't actually support Microsoft's attribute spellings currently and have no attributes there to document. I think the fixme should probably read "TODO: support documenting Microsoft spellings" or something more concrete.<br>
> Done. (I accidentally pushed the old version, so this is done in r341100.)<br>
><br>
> For what it's worth, we have one `Microsoft` spelling listed in the .td file already (but I assume this has no effect):<br>
><br>
> ```<br>
> def Uuid : InheritableAttr {<br>
>   let Spellings = [Declspec<"uuid">, Microsoft<"uuid">];<br>
> ```<br>
<br>
Hmm, I take it back, we do support a Microsoft attribute, only to warn<br>
about it being deprecated and telling users to use __declspec instead:<br>
<a href="https://godbolt.org/z/_0ZxWq" rel="noreferrer" target="_blank">https://godbolt.org/z/_0ZxWq</a><br>
<br>
I remember when we tried to add more support for parsing Microsoft<br>
attributes, but I had the impression we didn't support them beyond the<br>
very basics of parsing. Perhaps we do want to document them though,<br>
since there's at least one?<br></blockquote><div><br></div><div>Given that doing so will make the "supported syntaxes" table wider for all attributes, and it's already about as wide as seems reasonable, and we consider all attributes of this form to be deprecated, I don't think it's worth it. Maybe if we only included non-empty columns in the syntax table?</div></div></div>