<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 15, 2015 at 2:46 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">On Wed, Jul 15, 2015 at 5:37 PM, Ristow, Warren<br>
<<a href="mailto:warren_ristow@playstation.sony.com">warren_ristow@playstation.sony.com</a>> wrote:<br>
>> Is there a reason you cannot use -fms-extensions instead?  ...<br>
><br>
> We don't want to enable support of _all_ MS extensions, so just directly using -fms-extensions isn't viable for us.  You mention __declspec isn't a Sony extension, but really a "Microsoft (and Borland)" extension.  Along that line, I'd say one way of looking at this is saying it will now be a "Microsoft (and Borland and Sony)" extension.<br>
<br>
</span>I think you missed my point, but I take your meaning just the same.<br>
Thank you for the explanation, it's what I figured the case would be.<br>
<br>
Sony would like support for __declspec without turning on all MS<br>
extensions. We currently have the same idea in place to support<br>
__declspec for CUDA. It seems to me that there are cases where<br>
unconditional support of __declspec is desired. I am wondering if we<br>
want an -fdeclspec-ext (or something) that enables just __declspec<br>
support, instead of trying to tie this extension to compiler vendors.<br>
Obviously, we could turn this extension on for Sony's triple as well<br>
in that case. I'm wondering if that's a design we'd want to consider<br>
exploring or not.<br></blockquote><div><br></div><div>Perhaps we should rethink r238238.  The change was definitely contentious.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class=""><div class="h5"><br>
~Aaron<br>
<br>
><br>
> -Warren<br>
><br>
> -----Original Message-----<br>
> From: <a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a> [mailto:<a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a>] On Behalf Of Aaron Ballman<br>
> Sent: Wednesday, July 15, 2015 11:48 AM<br>
> To: Ristow, Warren<br>
> Cc: <a href="mailto:reviews%2BD11207%2Bpublic%2B3fbdfe66c3b43ce8@reviews.llvm.org">reviews+D11207+public+3fbdfe66c3b43ce8@reviews.llvm.org</a>; Anton Korobeynikov; llvm cfe<br>
> Subject: Re: [PATCH] D11207: Enable recognition of __declspec for PS4<br>
><br>
> On Wed, Jul 15, 2015 at 2:07 PM, Ristow, Warren<br>
> <<a href="mailto:warren_ristow@playstation.sony.com">warren_ristow@playstation.sony.com</a>> wrote:<br>
>>> Can I get a brief description of what you're trying to accomplish and<br>
>>> why this is needed?<br>
>><br>
>> We intend to support __declspec(dllimport)/__declspec(dllexport) on PS4, and we will be uploading code reviews for that.  So we're enabling recognition of __declspec on PS4 as a prerequisite for that.<br>
><br>
> Is there a reason you cannot use -fms-extensions instead? I'm a little<br>
> uncomfortable with the line being drawn here because this isn't a Sony<br>
> extension, it's a Microsoft (and Borland) one where we are trying to<br>
> be compatible with what those compilers do. For instance, this opens<br>
> the door for Sony-specific behavior of a Microsoft compiler feature<br>
> that Clang supports. However, I may also be out of the loop on why<br>
> this is needed. :-)<br>
><br>
> ~Aaron<br>
><br>
>><br>
>> -Warren<br>
>><br>
>> -----Original Message-----<br>
>> From: <a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a> [mailto:<a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a>] On Behalf Of Aaron Ballman<br>
>> Sent: Wednesday, July 15, 2015 10:49 AM<br>
>> To: <a href="mailto:reviews%2BD11207%2Bpublic%2B3fbdfe66c3b43ce8@reviews.llvm.org">reviews+D11207+public+3fbdfe66c3b43ce8@reviews.llvm.org</a><br>
>> Cc: Ristow, Warren; Anton Korobeynikov; llvm cfe<br>
>> Subject: Re: [PATCH] D11207: Enable recognition of __declspec for PS4<br>
>><br>
>> On Tue, Jul 14, 2015 at 7:36 PM, Warren Ristow<br>
>> <<a href="mailto:warren_ristow@playstation.sony.com">warren_ristow@playstation.sony.com</a>> wrote:<br>
>>> wristow created this revision.<br>
>>> wristow added a reviewer: asl.<br>
>>> wristow added a subscriber: cfe-commits.<br>
>>><br>
>>> This change includes the changes of <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11164&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=k8Yi55tW7yMKuwxeplUYLoxLze0FK3CKOOuwvTfvy9w&s=YkIPhg3p0I5TEOQtBJAQaA2xdlnkql1qGzsQSZ5M4Y4&e=" rel="noreferrer" target="_blank">reviews.llvm.org/D11164</a> (reserving a flag for Sony/PS extensions), and also uses that flag to enable recognition of __declspec.  This supersedes D11164.<br>
>>><br>
>>> Can someone review, and if OK then commit this patch for me, please?<br>
>><br>
>> Can I get a brief description of what you're trying to accomplish and<br>
>> why this is needed?<br>
>><br>
>> ~Aaron<br>
>><br>
>>><br>
>>> -Warren Ristow<br>
>>> SN Systems - Sony Computer Entertainment Group<br>
>>><br>
>>> <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11207&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=k8Yi55tW7yMKuwxeplUYLoxLze0FK3CKOOuwvTfvy9w&s=GthBeK4iR7HCR7N7FsGl83r2jdKLrl6Gp1sfWy88li0&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11207</a><br>
>>><br>
>>> Files:<br>
>>>   include/clang/Basic/LangOptions.def<br>
>>>   include/clang/Basic/TokenKinds.def<br>
>>>   include/clang/Driver/CC1Options.td<br>
>>>   include/clang/Parse/Parser.h<br>
>>>   lib/Basic/IdentifierTable.cpp<br>
>>>   lib/Driver/Tools.cpp<br>
>>>   lib/Frontend/CompilerInvocation.cpp<br>
>>>   lib/Parse/ParseDecl.cpp<br>
>>>   test/Lexer/keywords_test.c<br>
>>>   test/Lexer/keywords_test.cpp<br>
>>><br>
>>><br>
>>> _______________________________________________<br>
>>> cfe-commits mailing list<br>
>>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
>>><br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>