[PATCH] D125649: [ARM SEH 7] [WIP] [ARM] Adjust the frame pointer when it's needed for SEH unwinding

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 15 22:30:08 PDT 2022


mstorsjo added a comment.

In D125649#3514665 <https://reviews.llvm.org/D125649#3514665>, @efriedma wrote:

> Can we just rearrange the order we save registers on the stack?  I mean, generating an extra push instruction just to satisfy the unwinder is sort of silly, but it seems like the least bad alternative.  (And we only need to do it in cases involving dynamic allocation.)
>
>   push {r4-r5}
>   push {r11, lr}
>   mov r11, sp

Oh, thanks, that sounds like a good idea. I'll see how that turns out!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125649/new/

https://reviews.llvm.org/D125649



More information about the llvm-commits mailing list