Thanks I'll fix that. -----Original Message----- From: Fedor Sergeev via Phabricator [mailto:revi...@reviews.llvm.org] Sent: Tuesday, June 13, 2017 3:30 PM To: Blower, Melanie <melanie.blo...@intel.com>; zhangsheng...@huawei.com; olivier...@gmail.com; kalinichev.s...@gmail.com; kf...@kde.org; m...@milianw.de; cfe-commits@lists.llvm.org; Keane, Erich <erich.ke...@intel.com> Cc: kli...@google.com; simon.dar...@imgtec.com; anastasia.stul...@arm.com; arichardson....@gmail.com; fedor.serg...@oracle.com Subject: [PATCH] D34158: to support gcc 4.8 (and newer) compatibility on Linux, preinclude <stdc-predef.h>
fedor.sergeev added a comment. There is no /usr/include/stdc-predef.h on Solaris and I see no sense in adding this -include for anything except Linux (as docs are rather clear that this functionality is Linux-only). Checked gcc on Solaris11/12 both SPARC and X86 - none are doing stdc-predef.h inclusion (checking with gcc - -dI </dev/null -E | grep stdc-predef ) https://reviews.llvm.org/D34158 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
