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

[Eric Christopher]

On Mar 3, 2010, at 3:24 PM, Chris Lattner wrote:

> 
> 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

Oh? People couldn't find it using clang which is why I added. I'll remove that part though.

-eric