From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-3.0 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL autolearn=ham autolearn_force=no version=3.4.2 Received: from mother.openwall.net (mother.openwall.net [195.42.179.200]) by inbox.vuxu.org (OpenSMTPD) with SMTP id 90f30f67 for ; Wed, 22 Jan 2020 01:32:09 +0000 (UTC) Received: (qmail 30380 invoked by uid 550); 22 Jan 2020 01:32:07 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-ID: Reply-To: musl@lists.openwall.com Received: (qmail 30362 invoked from network); 22 Jan 2020 01:32:07 -0000 Date: Tue, 21 Jan 2020 20:31:55 -0500 From: Rich Felker To: musl@lists.openwall.com Message-ID: <20200122013155.GW30412@brightrain.aerifal.cx> References: <20200121185215.5958-1-armccurdy@gmail.com> <20200122003615.GV30412@brightrain.aerifal.cx> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: Rich Felker Subject: Re: [musl] [PATCH v2] Add big-endian support to ARM assembler memcpy On Tue, Jan 21, 2020 at 04:47:42PM -0800, Andre McCurdy wrote: > On Tue, Jan 21, 2020 at 4:36 PM Rich Felker wrote: > > On Tue, Jan 21, 2020 at 10:52:15AM -0800, Andre McCurdy wrote: > > > Allow the existing ARM assembler memcpy implementation to be used for > > > both big and little endian targets. > > > --- > > > > > > Exactly the same changes as before but rebased to account for > > > whitespace changes in the preceding patch to add Thumb2 support. > > > > Thanks. I'm not sure if I'll ever apply this, since I think there's a > > better memcpy we may be adopting, but it's nice to have in case we > > want to. > > OK. Slightly disappointed to hear that. I've already been carrying > this patch for many months in my own builds so hopefully the new > memcpy() which will allow it to be dropped is imminent! If it doesn't look immediately clear that we should go with the new one, I'll go ahead and merge this after release. I wasn't aware anyone was using or cared about BE, but if you do I don't want to hold it back just because it might no longer be relevant in future. Rich