This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] powerpc: Add a POWER8-optimized version of expf()
- From: "Paul E. Murphy" <murphyp at linux dot vnet dot ibm dot com>
- To: Tulio Magno Quites Machado Filho <tuliom at linux dot vnet dot ibm dot com>, libc-alpha at sourceware dot org
- Cc: munroesj at linux dot vnet dot ibm dot com
- Date: Mon, 27 Jun 2016 09:11:02 -0500
- Subject: Re: [PATCH] powerpc: Add a POWER8-optimized version of expf()
- Authentication-results: sourceware.org; auth=none
- References: <1466715228-29670-1-git-send-email-tuliom at linux dot vnet dot ibm dot com> <f4acab4b-b7b2-21d0-4676-f3320089c715 at linux dot vnet dot ibm dot com> <87furyvjsc dot fsf at totoro dot br dot ibm dot com>
On 06/27/2016 08:33 AM, Tulio Magno Quites Machado Filho wrote:
> "Paul E. Murphy" <murphyp@linux.vnet.ibm.com> writes:
>> Similarly, the formatting seems inconsistent with
>> the other PPC asm.
>
> Could you give me more details, please?
>
I.e randomly choosing an existing file,
sysdeps/powerpc/powerpc64/power8/strcmp.S
Primarily, it is the spacing between operands and operations.
Almost all assembler files I've come across put a tab between
the opcode and the first operand. This patch uses spaces
exclusively.