[PATCH] D18708: Set C99 as default C Standard for PS4 target

[Richard Smith via cfe-commits]

On Fri, Apr 1, 2016 at 6:43 PM, Douglas Yung <
Douglas_Yung at playstation.sony.com> wrote:

> dyung added a comment.
>
> In http://reviews.llvm.org/D18708#390166, @rsmith wrote:
>
> > In http://reviews.llvm.org/D18708#390150, @dyung wrote:
> >
> > > From my understanding, there are 2 issues that block us. The first is
> that we currently do not ship all of the header files for C11. The second
> is that we do not yet fully have C11 library support yet for our platform.
> >
> >
> > How do things go wrong if Clang is used in C11 mode against a C99
> library? Do you have code that conditionally uses C11 library features if
> they're available? (And thanks for explaining, by the way; this is useful
> information for when we next consider changing the default language mode --
> we'll likely be changing the default C++ language mode soon.)
> >
> > Anyway, it seems reasonable for the PS4 toolchain to control its
> defaults here, and the patch itself LGTM.
>
>
> My understanding is that we currently do not ship all of the headers
> required for C11, so if users tried to use anything from those, they would
> be unable to do so. I am also under the impression that C11 requires some
> library support which we do not yet provide.


OK, but why is it a problem for the compiler's C11 mode to be enabled? I'm
guessing that something somewhere has code like

  #if __STDC_VERSION__ >= 201112L
  #include <some_c11_header.h>
  #endif

Is that the problem? If not, what? Is our C11 mode rejecting valid C99 code
somehow?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160404/82833f4d/attachment.html>