diff options
| author | Erich Keane <erich.keane@intel.com> | 2017-11-20 17:57:42 +0000 |
|---|---|---|
| committer | Erich Keane <erich.keane@intel.com> | 2017-11-20 17:57:42 +0000 |
| commit | 5c086c7626fab2617c0178a3951378bcbce2271c (patch) | |
| tree | e073f8fc692b20d4dfcd8aead20817d8bf883db1 /clang/lib/Driver/Job.cpp | |
| parent | 991447d6f892f4763347528290093c10806ece74 (diff) | |
| download | bcm5719-llvm-5c086c7626fab2617c0178a3951378bcbce2271c.tar.gz bcm5719-llvm-5c086c7626fab2617c0178a3951378bcbce2271c.zip | |
For Linux/gnu compatibility, preinclude <stdc-predef.h> if the file is available
As reported in llvm bugzilla 32377.
Here’s a patch to add preinclude of stdc-predef.h.
The gcc documentation says “On GNU/Linux, <stdc-predef.h> is pre-included.”
See https://gcc.gnu.org/gcc-4.8/porting_to.html;
The preinclude is inhibited with –ffreestanding.
Basically I fixed the failing test cases by adding –ffreestanding which inhibits
this behavior.
I fixed all the failing tests, including some in extra/test, there's a separate
patch for that which is linked here
Patch By: mibintc
Differential Revision: https://reviews.llvm.org/D34158
llvm-svn: 318669
Diffstat (limited to 'clang/lib/Driver/Job.cpp')
| -rw-r--r-- | clang/lib/Driver/Job.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Driver/Job.cpp b/clang/lib/Driver/Job.cpp index 765c05752d8..b494bc75282 100644 --- a/clang/lib/Driver/Job.cpp +++ b/clang/lib/Driver/Job.cpp @@ -64,7 +64,7 @@ static bool skipArgs(const char *Flag, bool HaveCrashVFS, int &SkipNum, .Cases("-internal-externc-isystem", "-iprefix", true) .Cases("-iwithprefixbefore", "-isystem", "-iquote", true) .Cases("-isysroot", "-I", "-F", "-resource-dir", true) - .Cases("-iframework", "-include-pch", true) + .Cases("-iframework", "-include-pch", "-fsystem-include-if-exists", true) .Default(false); if (IsInclude) return HaveCrashVFS ? false : true; |
