Sounds pretty nice to me. For what it's worth, I'm testing a sigtramp
unwinder on MIPS/Linux that could almost but not quite use this:
+struct mips_prologue_cache
+{
+ /* The stack pointer at the time this frame was created; i.e. the
+ caller's stack pointer when this function was called. It is used
+ to identify this frame. */
+ CORE_ADDR prev_sp;
+
+ CORE_ADDR tramp_start;
+
+ int kind;
+
+ /* Saved register offsets. */
+ struct trad_frame_saved_reg *saved_regs;
+};
(so that the frame ID is constant for both instructions of the
trampoline).