[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAH9TF6PrMGvhAWS-HDdWyZA-QnyjwiwBem-XMV1-eN2LvNUndg@mail.gmail.com>
Date: Sat, 22 Feb 2025 01:40:25 +0100
From: Alex Rønne Petersen <alex@...xrp.com>
To: Rich Felker <dalias@...c.org>
Cc: musl@...ts.openwall.com
Subject: Re: [PATCH] i386, x86_64: Set the symbol type for the START function.
On Sat, Feb 22, 2025 at 1:30 AM Rich Felker <dalias@...c.org> wrote:
>
> On Sat, Nov 23, 2024 at 01:19:51AM +0100, Alex Rønne Petersen wrote:
> > As for other architectures. Noticed this while running Zig's gen_stubs.zig tool.
> > ---
> > arch/i386/crt_arch.h | 1 +
> > arch/x86_64/crt_arch.h | 1 +
> > 2 files changed, 2 insertions(+)
> >
> > diff --git a/arch/i386/crt_arch.h b/arch/i386/crt_arch.h
> > index 43c8477a..1a80fce3 100644
> > --- a/arch/i386/crt_arch.h
> > +++ b/arch/i386/crt_arch.h
> > @@ -3,6 +3,7 @@ __asm__(
> > ".weak _DYNAMIC \n"
> > ".hidden _DYNAMIC \n"
> > ".global " START "\n"
> > +".type " START ",%function \n"
> > START ":\n"
> > " xor %ebp,%ebp \n"
> > " mov %esp,%eax \n"
> > diff --git a/arch/x86_64/crt_arch.h b/arch/x86_64/crt_arch.h
> > index 3eec61bd..b1c9c476 100644
> > --- a/arch/x86_64/crt_arch.h
> > +++ b/arch/x86_64/crt_arch.h
> > @@ -1,6 +1,7 @@
> > __asm__(
> > ".text \n"
> > ".global " START " \n"
> > +".type " START ",%function \n"
> > START ": \n"
> > " xor %rbp,%rbp \n"
> > " mov %rsp,%rdi \n"
> > --
> > 2.40.1
>
> Looks ok but was missing x32. I'll add it when merging.
Oops, I sent a v2 of the patch that includes x32, but didn't send it
as a reply to this one, sorry.
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
