This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH][PPC] Remove unnecessary local variable mptwo
- From: Carlos O'Donell <carlos at systemhalted dot org>
- To: Siddhesh Poyarekar <siddhesh at redhat dot com>
- Cc: libc-alpha at sourceware dot org, rsa at us dot ibm dot com
- Date: Thu, 10 Jan 2013 10:50:11 -0500
- Subject: Re: [PATCH][PPC] Remove unnecessary local variable mptwo
- References: <20130109114033.GC2692@spoyarek.pnq.redhat.com>
On 01/09/2013 06:40 AM, Siddhesh Poyarekar wrote:
> Hi,
>
> Attached patch removes the local variable mptwo from the __inv
> function so that it uses the global static definition instead. No
> regressions resulting from this patch. OK to commit?
>
> Siddhesh
>
> * sysdeps/powerpc/powerpc32/power4/fpu/mpa.c (__inv): Remove
> local variable MPTWO.
> * sysdeps/powerpc/powerpc64/power4/fpu/mpa.c (__inv): Likewise.
How does this impact the generated code?
How does that impact mpatan's performance?
Cheers,
Carlos.