Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sun, 8 Aug 2021 19:05:36 -0400
From: Rich Felker <dalias@...c.org>
To: Jon Chesterfield <jonathanchesterfield@...il.com>
Cc: musl@...ts.openwall.com, Olivier Galibert <galibert@...ox.com>
Subject: Re: [PATCH] stdlib: implement qsort_r

On Sun, Aug 08, 2021 at 10:21:06PM +0100, Jon Chesterfield wrote:
> On Sun, Aug 8, 2021 at 9:39 PM √Črico Nogueira <ericonr@...root.org> wrote:
> 
> > This is the favored approach, from what I understood of the discussions
> > surrounding it. It's implemented with musl's namespacing rules and such
> > in [1].
> >
> 
> Unsurprisingly I like your patch.
> 
> >
> > It is badly optimized for some archs, unfortunately, as explained in the
> > thread from [2]. I believe that's what's holding it up.
> >
> > [1]
> > https://inbox.vuxu.org/musl/20210309210213.29539-1-ericonr@disroot.org/
> > [2]
> > https://inbox.vuxu.org/musl/20210309150320.GU32655@brightrain.aerifal.cx/
> 
> 
> And that's what I get for not reading the list carefully enough. I missed
> the thread
> from March entirely.

No problem. Indeed, this was stalled because of concerns about bad
compiler behavior on some archs, but I really don't want to allow that
to dictate how we solve the problem. The compiler is doing the wrong
thing (failing to tail call) and should just be fixed. I'll probably
merge the tail-call patch as-is, or with any minimal fixes needed,
right after this release so as not to make major API changes at last
minute.

Rich

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.