[PATCH] D45383: Limit types of builtins that can be redeclared.

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 9 06:27:08 PDT 2018

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.



More information about the cfe-commits mailing list