[cfe-commits] r97666 - /cfe/trunk/lib/Frontend/InitHeaderSearch.cpp

[Charles Davis]

On 3/3/10 2:41 PM, Eric Christopher wrote:
> Author: echristo
> Date: Wed Mar  3 15:41:50 2010
> New Revision: 97666
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=97666&view=rev
> Log:
> Add in more c++ header paths for later gccs under gentoo linux.
> Add in c header path for various linuxes as well.
> 
> Partial patch from Christian Adåker!
> 
> Modified:
>     cfe/trunk/lib/Frontend/InitHeaderSearch.cpp
> 
> Modified: cfe/trunk/lib/Frontend/InitHeaderSearch.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitHeaderSearch.cpp?rev=97666&r1=97665&r2=97666&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Frontend/InitHeaderSearch.cpp (original)
> +++ cfe/trunk/lib/Frontend/InitHeaderSearch.cpp Wed Mar  3 15:41:50 2010
> @@ -432,6 +432,9 @@
>    case llvm::Triple::MinGW32:
>      AddPath("c:/mingw/include", System, true, false, false);
>      break;
> +  case llvm::Triple::Linux:
> +    AddPath("/usr/include/linux", System, false, false, false);
You sure that's right?

That directory contains the Linux kernel headers. Every app I've ever
seen that uses those headers always includes them like so:

#include <linux/header.h>

I don't think that should be in the default include path.

Chip