This is an archive of the discontinued LLVM Phabricator instance.

[llvm][AArch64] Fix "+all" feature for sysreg aliases
ClosedPublic

Authored by DavidSpickett on Jul 5 2022, 9:17 AM.

Details

Summary

For example the predres extension adds one instruction that
is a sys alias. Previously this wasn't disassembled properly
with "+all".

This was because a check for "+all" was added to haveFeatures
in AArch64SysReg but not in SysAlias.

Diff Detail

Event Timeline

DavidSpickett created this revision.Jul 5 2022, 9:17 AM
DavidSpickett requested review of this revision.Jul 5 2022, 9:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 5 2022, 9:17 AM

clang-format

lenary accepted this revision.Jul 5 2022, 9:32 AM
This revision is now accepted and ready to land.Jul 5 2022, 9:32 AM
MaskRay accepted this revision.Jul 5 2022, 12:00 PM
This revision was landed with ongoing or failed builds.Jul 6 2022, 1:42 AM
This revision was automatically updated to reflect the committed changes.