beanz added a comment.

Perhaps naming this warning 'incompatible-sdk' isn't the best name. I can see 
how it would cause some confusion for Visual Studio users. On Darwin the SDK is 
the collection of headers and libraries you link against, not the toolkit you 
have installed.

Perhaps calling the warning 'incompatible-sysroot' is a better name. Even 
though this patch only implements it for Darwin I imagine that similar 
functionality could be added for Windows and Linux too. The idea being to have 
the compiler perform some basic check to ensure that the headers and libraries 
you are building against actually support your target.

Updated patches coming shortly.


http://reviews.llvm.org/D18088



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to