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

[Chris Lattner]

On Mar 3, 2010, at 2:19 PM, Eric Christopher wrote:

>>> 
>> 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.
> 
> It also contains headers like... stddef.h.
> 
> I guess we could use the internal gcc one, but I'd prefer to use broader headers so we could avoid the need to do that.

Yeah, I don't think this is the right fix, it will cause user code to start getting kernel versions of stuff when they don't mean to.  Clang provides its own stddef.h