[PATCH] D135276: [AArch64] Add support for the SEH opcode for return address signing
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 5 14:01:20 PDT 2022
mstorsjo added a comment.
In D135276#3837965 <https://reviews.llvm.org/D135276#3837965>, @efriedma wrote:
> As far as I know, there isn't any publicly released hardware running Windows that supports PAC.
Yeah, that's my understanding too.
(I kinda wondered that you might have access to such HW though, but if you have, you're of course also not allowed to comment on it. Although for this case, it wouldn't really reveal anything about the HW itself, other than that it exists, only about the Windows unwinder implementation.)
The reason I'm asking is for assessing the impact of the bug that we're currently willingly generated PAC instructions without matching unwind info. In https://code.videolan.org/videolan/vlc/-/merge_requests/2662 I suggested stopping using `-mbranch-protection=standard` for Windows targets (so current binaries won't start misbehaving on future HW) until this is implemented properly in LLVM. However, I'm getting counterarguments for preferring keeping the added security (even though no current HW benefits from it, only future HW), and that the lacking unwind info shouldn't have any notable runtime effect.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135276/new/
https://reviews.llvm.org/D135276
More information about the llvm-commits
mailing list