Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200115154553.GH30412@brightrain.aerifal.cx>
Date: Wed, 15 Jan 2020 10:45:53 -0500
From: Rich Felker <dalias@...c.org>
To: musl@...ts.openwall.com
Subject: Re: [PATCH 2/2] Add big-endian support to ARM assembler memcpy

On Fri, Sep 13, 2019 at 01:38:34PM -0700, Andre McCurdy wrote:
> On Fri, Sep 13, 2019 at 11:59 AM Rich Felker <dalias@...c.org> wrote:
> >
> > On Fri, Sep 13, 2019 at 11:44:32AM -0700, Andre McCurdy wrote:
> > > Allow the existing ARM assembler memcpy implementation to be used for
> > > both big and little endian targets.
> >
> > Nice. I don't want to merge this just before release, but as long as
> > it looks ok I should be able to review and merge it afterward.
> >
> > Note that I'd really like to replace this giant file with C using
> > inline asm just for the inner block copies and C for all the flow
> > control, but I don't mind merging this first as long as it's correct.
> 
> Sounds good. I'll wait for your feedback after the upcoming release.

Sorry this dropped off my radar. I'd like to merge at least the thumb
part since it's simple enough to review quickly and users have
actually complained about memcpy being slow on armv7 with -mthumb as
default.

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.