On Wed, Jul 15, 2015 at 2:07 PM, Ristow, Warren <warren_ris...@playstation.sony.com> wrote: >> Can I get a brief description of what you're trying to accomplish and >> why this is needed? > > 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.
Is there a reason you cannot use -fms-extensions instead? I'm a little uncomfortable with the line being drawn here because this isn't a Sony extension, it's a Microsoft (and Borland) one where we are trying to be compatible with what those compilers do. For instance, this opens the door for Sony-specific behavior of a Microsoft compiler feature that Clang supports. However, I may also be out of the loop on why this is needed. :-) ~Aaron > > -Warren > > -----Original Message----- > From: aaron.ball...@gmail.com [mailto:aaron.ball...@gmail.com] On Behalf Of > Aaron Ballman > Sent: Wednesday, July 15, 2015 10:49 AM > To: reviews+d11207+public+3fbdfe66c3b43...@reviews.llvm.org > Cc: Ristow, Warren; Anton Korobeynikov; llvm cfe > Subject: Re: [PATCH] D11207: Enable recognition of __declspec for PS4 > > On Tue, Jul 14, 2015 at 7:36 PM, Warren Ristow > <warren_ris...@playstation.sony.com> wrote: >> wristow created this revision. >> wristow added a reviewer: asl. >> wristow added a subscriber: cfe-commits. >> >> This change includes the changes of reviews.llvm.org/D11164 (reserving a >> flag for Sony/PS extensions), and also uses that flag to enable recognition >> of __declspec. This supersedes D11164. >> >> Can someone review, and if OK then commit this patch for me, please? > > Can I get a brief description of what you're trying to accomplish and > why this is needed? > > ~Aaron > >> >> -Warren Ristow >> SN Systems - Sony Computer Entertainment Group >> >> http://reviews.llvm.org/D11207 >> >> Files: >> include/clang/Basic/LangOptions.def >> include/clang/Basic/TokenKinds.def >> include/clang/Driver/CC1Options.td >> include/clang/Parse/Parser.h >> lib/Basic/IdentifierTable.cpp >> lib/Driver/Tools.cpp >> lib/Frontend/CompilerInvocation.cpp >> lib/Parse/ParseDecl.cpp >> test/Lexer/keywords_test.c >> test/Lexer/keywords_test.cpp >> >> >> _______________________________________________ >> cfe-commits mailing list >> cfe-commits@cs.uiuc.edu >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> _______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits