[PATCH] D34158: For standards compatibility, preinclude <stdc-predef.h> if the file is available

Melanie Blower via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 28 08:44:22 PDT 2017


mibintc planned changes to this revision.
mibintc added a comment.

@erichkeane contacted me offline and pointed out I'm twine-ing with " not <. i'm planning to change the option name from "finclude if exists" to "fsystem include if exists", then i'll quote with < not ". hope to get this updated patch into review later today. let me know if you think i'm off track. thanks for all your careful review. i knew there was something wrong with the include like Fedor pointed out but i couldn't see where i went wrong.


Repository:
  rL LLVM

https://reviews.llvm.org/D34158





More information about the cfe-commits mailing list