Skip to content

Commit 90e97c5

Browse files
authored
Ignore DBGArchitecture from dsymForUUID's plist (#2219)
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 (cherry picked from commit 2cedc44)
1 parent ed25809 commit 90e97c5

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)