This is an archive of the discontinued LLVM Phabricator instance.

[LLDB] Add tests for PECOFF arm architecture identification
ClosedPublic

Authored by mstorsjo on Sep 24 2019, 2:24 AM.

Details

Summary

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.)

Diff Detail

Event Timeline

mstorsjo created this revision.Sep 24 2019, 2:24 AM
labath accepted this revision.Sep 24 2019, 4:47 AM
This revision is now accepted and ready to land.Sep 24 2019, 4:47 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 24 2019, 5:20 AM