From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3763 invoked by alias); 17 May 2013 00:32:41 -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 3753 invoked by uid 89); 17 May 2013 00:32:41 -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; Fri, 17 May 2013 00:32:41 +0000 Received: by topped-with-meat.com (Postfix, from userid 5281) id AD0002C084; Thu, 16 May 2013 17:32:39 -0700 (PDT) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit From: Roland McGrath To: Richard Henderson Cc: libc-ports@sourceware.org Subject: Re: [PATCH roland/arm-memcpy] ARM: Make multiarch memcpy always use NEON when compiler does In-Reply-To: Richard Henderson's message of Tuesday, 14 May 2013 08:47:39 -0700 <51925C9B.2030307@twiddle.net> References: <20130513224722.9F17C2C05E@topped-with-meat.com> <51925C9B.2030307@twiddle.net> Message-Id: <20130517003239.AD0002C084@topped-with-meat.com> Date: Fri, 17 May 2013 00:32: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=7GllbDADBFUA:10 a=Z6MIti7PxpgA:10 a=kj9zAlcOel0A:10 a=hOe2yjtxAAAA:8 a=hV1xvSDiiQEA:10 a=_Rx04MRSKW2cw1avR0gA:9 a=CjuIK1q_8ugA:10 X-SW-Source: 2013-05/txt/msg00078.txt.bz2 > I think you're better off leaving the __memcpy_neon symbol in all cases. > Having memcpy be an alias to __memcpy_neon would make the rest of the patch > simpler. I don't see what could be any simpler (or different) other than this one #ifdef in the test case.