From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18400 invoked by alias); 1 May 2013 16:50:13 -0000 Mailing-List: contact libc-ports-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: libc-ports-owner@sourceware.org Received: (qmail 18384 invoked by uid 89); 1 May 2013 16:50:12 -0000 X-Spam-SWARE-Status: No, score=-3.0 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,TW_CP autolearn=ham version=3.3.1 Received: from toast.topped-with-meat.com (HELO topped-with-meat.com) (204.197.218.159) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Wed, 01 May 2013 16:50:12 +0000 Received: by topped-with-meat.com (Postfix, from userid 5281) id E5BD92C098; Wed, 1 May 2013 09:50:10 -0700 (PDT) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit From: Roland McGrath To: Richard Henderson Cc: Will Newton , libc-ports@sourceware.org, patches@linaro.org Subject: Re: [PATCH, v5] ARM: Add Cortex-A15 optimized NEON and VFP memcpy routines, with IFUNC. In-Reply-To: Richard Henderson's message of Wednesday, 1 May 2013 13:52:59 +0100 <5181102B.8060006@twiddle.net> References: <517FF73E.5020509@linaro.org> <5181102B.8060006@twiddle.net> Message-Id: <20130501165010.E5BD92C098@topped-with-meat.com> Date: Wed, 01 May 2013 16:50:00 -0000 X-CMAE-Score: 0 X-CMAE-Analysis: v=2.1 cv=LYSvtFvi c=1 sm=1 tr=0 a=WkljmVdYkabdwxfqvArNOQ==:117 a=14OXPxybAAAA:8 a=cFfnH-CuOakA:10 a=Z6MIti7PxpgA:10 a=kj9zAlcOel0A:10 a=hOe2yjtxAAAA:8 a=_hi65yC6-YUA:10 a=68gmNxIcEcwRXSkqw7YA:9 a=CjuIK1q_8ugA:10 X-SW-Source: 2013-05/txt/msg00015.txt.bz2 > > + /* Jump directly into the sequence below at the correct offset. */ > > + add pc, pc, tmp1, lsl #1 > > This is going to fail for NaCl, defining ARM_ALWAYS_BX. I'm not quite > sure all of what needs to be done to fix that. I'm sure Roland will > have some advice... > > Perhaps the answer is "nothing" and NaCl should be vectored back to the > non-ifunc version of the code. No, I want the new stuff. But I'd intended to do NaCl fixups as follow-on patches after this goes in. I don't expect other people to be worrying about NaCl issues at this point. Thanks, Roland