Page MenuHomePhabricator

[AArch64] Add +pauth archictecture option, allowing the v8.3a pointer authentication extension.
ClosedPublic

Authored by MarkMurrayARM on Tue, Jan 5, 5:39 AM.

Diff Detail

Event Timeline

MarkMurrayARM created this revision.Tue, Jan 5, 5:39 AM
MarkMurrayARM requested review of this revision.Tue, Jan 5, 5:39 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptTue, Jan 5, 5:39 AM

Fixed the parent commit.

chill added reviewers: ab, chill.Tue, Jan 5, 6:21 AM
chill accepted this revision.Wed, Jan 6, 4:49 AM

LGTM

This revision is now accepted and ready to land.Wed, Jan 6, 4:49 AM
This revision was landed with ongoing or failed builds.Fri, Jan 8, 5:22 AM
This revision was automatically updated to reflect the committed changes.
fhahn added a subscriber: fhahn.Fri, Jan 8, 7:06 AM

FWIW I think it would be good to have a bit more details in the description for changes such as this, like a link to the public docs for the extension.

chill added a comment.Fri, Jan 8, 7:24 AM

FWIW I think it would be good to have a bit more details in the description for changes such as this, like a link to the public docs for the extension.

I'm sorry, I assumed this information was shared when the extension itself ((this patch just adds the command line options),
was added to LLVM three years ago in https://reviews.llvm.org/D36517

The public documentation is in the Armv8-A ARM at https://developer.arm.com/documentation/ddi0487/latest
in section "D5.1.5 Pointer authentication in AArch64 state"