This is an archive of the discontinued LLVM Phabricator instance.

[AArch64] Add missing "pauth" feature to the .arch_extension directive.
ClosedPublic

Authored by MarkMurrayARM on Jan 19 2021, 6:53 AM.

Diff Detail

Event Timeline

MarkMurrayARM created this revision.Jan 19 2021, 6:53 AM
MarkMurrayARM requested review of this revision.Jan 19 2021, 6:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 19 2021, 6:53 AM
john.brawn accepted this revision.Jan 19 2021, 7:35 AM
john.brawn added a subscriber: john.brawn.

LGTM.

This revision is now accepted and ready to land.Jan 19 2021, 7:35 AM

On second though, looking at D94971, do you want to add a test to llvm/test/MC/AArch64/directive-arch_extension-negative.s?

The negative extension case is funny as the instruction generates hints in a way that the test doesn't like. Its not the same pattern.

This revision was landed with ongoing or failed builds.Jan 20 2021, 3:58 AM
This revision was automatically updated to reflect the committed changes.