================
@@ -2598,15 +2656,20 @@ void Darwin::AddDeploymentTarget(DerivedArgList &Args) 
const {
         << TargetVariantTriple->str();
   }
 
-  if (const Arg *A = Args.getLastArg(options::OPT_isysroot)) {
+  if (SDKInfo) {
+    if (!SDKInfo->supportsPlatformInfo(PlatformAndVersion->getPlatformInfo())) 
{
+      getDriver().Diag(diag::warn_incompatible_sysroot)
----------------
cyndyishida wrote:

Could you delay checking for this until the target triple is computed? That 
way, you don't need to add as many helpers for handling pieces of what will 
become the use target triple, like the new comparator, which looks identical to 
the one already for comparing `Triple` and `SDKPlatformInfo` and 
`getPlatformAndEnvironmentString`.

https://github.com/llvm/llvm-project/pull/176541
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to