On Thu, Jan 27, 2022 at 12:47 AM Richard Henderson < richard.hender...@linaro.org> wrote:
> On 1/25/22 12:29, Warner Losh wrote: > > setup_frame sets up a signalled stack frame. Associated routines to > > extract the pointer to the stack frame and to support alternate stacks. > > > > Signed-off-by: Stacey Son<s...@freebsd.org> > > Signed-off-by: Kyle Evans<kev...@freebsd.org> > > Signed-off-by: Warner Losh<i...@bsdimp.com> > > --- > > bsd-user/main.c | 5 +++ > > bsd-user/qemu.h | 3 +- > > bsd-user/signal.c | 82 +++++++++++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 89 insertions(+), 1 deletion(-) > > Reviewed-by: Richard Henderson <richard.hender...@linaro.org> > > > > +#if defined(TARGET_ARM) > > + return (sp - frame_size) & ~7; > > +#elif defined(TARGET_AARCH64) > > + return (sp - frame_size) & ~15; > > +#else > > + return sp - frame_size; > > +#endif > > Just double-checking that this is still in the cleanup queue. > I would expect x86 to require 16 byte alignment as well, for sse I'll add /* TODO: make this a target_arch function / define */ to make it clear that it's still on the list... Thanks for the reminder. There's a lot on the list, alas... Warner > > r~ >
