[cfe-commits] [PATCH] Warn if -isysroot is given an invalid path

Argyrios Kyrtzidis akyrtzi at gmail.com
Fri Dec 14 10:28:34 PST 2012


On Dec 14, 2012, at 9:55 AM, Bob Wilson <bob.wilson at apple.com> wrote:

> 
> On Dec 14, 2012, at 9:34 AM, Eric Christopher <echristo at gmail.com> wrote:
> 
>> 
>> 
>> 
>> On Fri, Dec 14, 2012 at 9:29 AM, Jordan Rose <jordan_rose at apple.com> wrote:
>>>> I can see bootstrapping scenarios such that you wouldn't want it. Possible to only put this warning into -Weverything and not into any other warning level? (TBH I didn't look at the patch so you may have done this…)
>>> 
>>> I didn't put it under -Weverything, but that's a good suggestion.  Thanks, Eric.
>> 
>> All warnings are under -Weverything. I think what Eric meant is that this should be DefaultIgnore.
>> 
>> 
>> Um yeah, this. :)
> 
> I don't think it's worth doing if it is only enabled with -Weverything.  It might be better to just abandon this.

As Jordan said, it can be off by default and enabled with "-Wmissing-sysroot".

> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121214/45bc9407/attachment.html>


More information about the cfe-commits mailing list