<br><br><div class="gmail_quote">On Fri, Dec 14, 2012 at 7:41 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>Sure, but I just don't see that providing much value.  The point of adding a warning was to help catch things like typos in the sysroot path.  What are the chances that someone is going to mis-type the sysroot and also specify -Wmissing-sysroot?  I'm assuming that typos are most likely to come up when someone is running the compiler by hand, rather than from a makefile.  I suppose that it could still help when a makefile specifies an SDK that is not currently installed.  I'm just not convinced that's worth the effort of adding and maintaining a new warning, but I'm open to being convinced.</div>

</div></blockquote></div><br></div></div><div class="gmail_extra">My only concerns are false positives in fairly standard use cases. I think the warning could be useful when debugging problems and if you know you aren't in the use cases that might give a false positive you can turn it on by default (perhaps with a new umbrella?) but false positives are bad for everyone :)</div>
<span class="HOEnZb"><font color="#888888">
<div class="gmail_extra"><br></div><div class="gmail_extra">-eric</div>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br>Actually maybe the best way is in putting this warning in 
-Wmissing-sysroot AND make  
-Wmissing-sysroot part of 
-Wunused-command-line-argument.<br><br>=> if you turn on -Wunused-command-line-argument you get warned if the flag is ignored because of an invalid path.<br><br>=> if you are warned and you do not care, you can always use -Wunused-command-line-argument -Wno-missing-sysroot<br>
<br>In this scenario, -Wmissing-sysroot is not of much use by itself, but its negative form is.<br><br>-- Matthieu<br><br>