This is an archive of the discontinued LLVM Phabricator instance.

[AArch64] Support llvm-mc/llvm-objdump -M no-aliases
ClosedPublic

Authored by MaskRay on May 23 2021, 11:08 PM.

Details

Summary

This enables the no-aliases forms of many instructions.

Depends on D103004

Diff Detail

Unit TestsFailed

Event Timeline

MaskRay created this revision.May 23 2021, 11:08 PM
MaskRay requested review of this revision.May 23 2021, 11:08 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 23 2021, 11:08 PM
MaskRay retitled this revision from [AArch64] Support llvm-mc/llvm-objdump -M no-aliases This enables the no-aliases forms of many instructions. Depends on D103004 to [AArch64] Support llvm-mc/llvm-objdump -M no-aliases.May 23 2021, 11:08 PM
MaskRay edited the summary of this revision. (Show Details)
tmatheson accepted this revision.May 26 2021, 4:35 AM
tmatheson added a subscriber: tmatheson.

LGTM, I will leave it to the RISC-V people to approve the predecessor

llvm/test/MC/AArch64/alias-addsubimm.s
2

Why change the triple here?

This revision is now accepted and ready to land.May 26 2021, 4:35 AM
MaskRay updated this revision to Diff 348019.May 26 2021, 10:50 AM

Prefer -mtriple=aarch64 (generic ELF) to a Linux oriented triple.
Using a generic ELF triple emphasizes that the test is portable across various ELF OSes not just Linux glibc.

MaskRay marked an inline comment as done.May 26 2021, 10:50 AM
This revision was landed with ongoing or failed builds.May 26 2021, 1:35 PM
This revision was automatically updated to reflect the committed changes.