[PATCH] D125536: [MC] [MCWin64EH] Try writing an ARM64 "packed epilog" even if the epilog doesn't share opcodes with the prolog
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat May 14 14:50:35 PDT 2022
mstorsjo added inline comments.
================
Comment at: llvm/lib/MC/MCWin64EH.cpp:1029
// FIXME: We should share epilog codes across epilogs, where possible,
// which would make this issue show up less frequently.
if (CodeWords > 0xFF || EpilogCount > 0xFFFF)
----------------
efriedma wrote:
> Is the fixme about sharing epilog codes fixed?
Yeah, I think so. (We only share epilogs if they're a perfect match, while we should be able to share them if one is a subset of the other one. But I think the potential benefit from that is rather small.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125536/new/
https://reviews.llvm.org/D125536
More information about the llvm-commits
mailing list