This is the mail archive of the mailing list for the glibc project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][PPC] Remove unnecessary local variable mptwo

On Thu, Jan 10, 2013 at 9:50 AM, Carlos O'Donell
<> wrote:
> 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?

Preventing regressions in performance of mpa is certainly something we
(IBM) are interested in.  In-fact we'd like to see improvements in
performance of this code if possible.

I understand that these cleanups are ultimately 'a good thing' and we
do have time to measure performance of the 2.17 release before we
release 2.18.

I think in order to have a good measure of the performance impacts we
need to make sure that Siddhesh's performance benchmark suite is in
place and that it can work with the 2.17 release so that we can get a


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]