caio/clang-p2996
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[LLDB] Add tests for PECOFF arm architecture identification

Browse Source 
Add a test case for the change from SVN r372657, and for the
preexisting ARM identification.

Add a missing ArchDefinitionEntry for PECOFF/arm64, and tweak
the ArmNt case to set the architecture to armv7 (ArmNt never ran
on anything lower than that). (This avoids a case where
ArchSpec::MergeFrom would override the arch from arm to armv7 and
ArchSpec::CoreUpdated would reset the OS to unknown at the same time.)

Differential Revision: https://reviews.llvm.org/D67951

llvm-svn: 372741
This commit is contained in:
Martin Storsjo
2019-09-24 12:20:52 +00:00
parent 5bb1525392
commit 544c8f48c8
4 changed files with 176 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp
View File

@@ -195,7 +195,7 @@ size_t ObjectFilePECOFF::GetModuleSpecifications(
specs.Append(module_spec);
break;
case MachineArmNt:
spec.SetTriple("arm-pc-windows");
spec.SetTriple("armv7-pc-windows");
specs.Append(module_spec);
break;
case MachineArm64: