erichkeane added a comment.

I'm curious why that file is declaring the builtin like that?  The declaration 
doesn't seem like it is ever useful, builtins are available without any 
includes/etc as far back as I could go on all 3 compilers that I validated 
against.

Additionally, I believe that the declaration is actually prohibited by the 
standard, since it is declaring an identifier that is reserved for the 
implementation.

Is this a situation where the target project cannot be corrected? That line 
simply needs to be removed as far as I can tell.


Repository:
  rL LLVM

https://reviews.llvm.org/D45383



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D45383: Limit... Alexander Richardson via Phabricator via cfe-commits
    • [PATCH] D45383: ... Erich Keane via Phabricator via cfe-commits

Reply via email to