[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