================
@@ -185,6 +185,12 @@ DarwinSDKInfo::parseDarwinSDKSettingsJSON(const 
llvm::json::Object *Obj) {
   if (!MaximumDeploymentVersion)
     return std::nullopt;
   PlatformInfoStorageType PlatformInfos = parsePlatformInfos(*Obj, *Version);
+  auto DisplayName = Obj->getString("DisplayName");
+  if (!DisplayName) {
+    // DisplayName should always be present, but most of the testing 
SDKSettings
----------------
cyndyishida wrote:

Is there a reason why the tests wouldn't be fixed? I don't think it makes sense 
to add logic purely for pre-dated tests.

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