Skip to content

Commit 2cedc44

Browse files
committed
Ignore DBGArchitecture from dsymForUUID's plist
When the architecture from the returned plist differs from the architecture lldb will pick when loading the binary file, lldb will reject the binary as not matching. We are working with UUID's in this case, so an architecture is not disambiguating anything; it just opens this possibility for failing to load the specified binary. Stop reading the architecture from the plist. <rdar://problem/71612561> Differential revision: https://reviews.llvm.org/D92692
1 parent 898d61b commit 2cedc44

File tree

2 files changed

+1
-8
lines changed

2 files changed

+1
-8
lines changed

lldb/source/Symbol/LocateSymbolFileMacOSX.cpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -342,13 +342,6 @@ static bool GetModuleSpecInfoFromUUIDDictionary(CFDictionaryRef uuid_dict,
342342
}
343343
}
344344

345-
cf_str = (CFStringRef)CFDictionaryGetValue((CFDictionaryRef)uuid_dict,
346-
CFSTR("DBGArchitecture"));
347-
if (cf_str && CFGetTypeID(cf_str) == CFStringGetTypeID()) {
348-
if (CFCString::FileSystemRepresentation(cf_str, str))
349-
module_spec.GetArchitecture().SetTriple(str.c_str());
350-
}
351-
352345
std::string DBGBuildSourcePath;
353346
std::string DBGSourcePath;
354347

lldb/test/API/macosx/lc-note/firmware-corefile/TestFirmwareCorefiles.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ def test_lc_note(self):
8585
'fi',
8686
'echo "<dict><key>$uuid</key><dict>"',
8787
'',
88-
'echo "<key>DBGArchitecture</key><string>x86_64</string>"',
88+
'echo "<key>DBGArchitecture</key><string>i386</string>"',
8989
'echo "<key>DBGDSYMPath</key><string>$dsym</string>"',
9090
'echo "<key>DBGSymbolRichExecutable</key><string>$bin</string>"',
9191
'echo "</dict></dict></plist>"',

0 commit comments

Comments
 (0)