This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] PowerPC: Optimized mpn functions for PowerPC64
- From: Adhemerval Zanella <azanella at linux dot vnet dot ibm dot com>
- To: will_schmidt at vnet dot ibm dot com
- Cc: "GNU C. Library" <libc-alpha at sourceware dot org>
- Date: Fri, 06 Dec 2013 15:49:28 -0200
- Subject: Re: [PATCH] PowerPC: Optimized mpn functions for PowerPC64
- Authentication-results: sourceware.org; auth=none
- References: <527795AB dot 20203 at linux dot vnet dot ibm dot com> <1386100479 dot 6272 dot 60 dot camel at brimstone>
Hi Will,
Thanks for the review.
On 03-12-2013 17:54, Will Schmidt wrote:
>
>> diff --git a/sysdeps/powerpc/powerpc64/addmul_1.S b/sysdeps/powerpc/powerpc64/addmul_1.S
>> new file mode 100644
>> index 0000000..f256506
>> --- /dev/null
>> +++ b/sysdeps/powerpc/powerpc64/addmul_1.S
>> @@ -0,0 +1,208 @@
>> +/* PowerPC64 __mpn_addmul_1 -- Multiply a limb vector with a limb and add
>> + the result to a second limb vector.
>> + Copyright (C) 1999-2013 Free Software Foundation, Inc.
>
> ^ from a glance at other files, I think this should be just "Copyright
> (C) 2013 ". Similar comment for the other files.
Since it is not really a new implementation, but rather an adapted on from other project
(MPN) I kept the original license start year (Joseph did it also for a e500 port).
>
> <snip..>
>
> I've read over most of the .asm, nothing jumps out at me as an issue or
> concern.
>
>
>> diff --git a/sysdeps/powerpc/powerpc64/submul_1.S b/sysdeps/powerpc/powerpc64/submul_1.S
>> new file mode 100644
>> index 0000000..145b1d4
>> --- /dev/null
>> +++ b/sysdeps/powerpc/powerpc64/submul_1.S
>> @@ -0,0 +1,21 @@
>> +/* PowerPC64 __mpn_addmul_1 -- Multiply a limb vector with a limb and subtract
>> + the result to a second limb vector.
>> + Copyright (C) 2013 Free Software Foundation, Inc.
>> + This file is part of the GNU C Library.
>> +
>> + The GNU C Library is free software; you can redistribute it and/or
>> + modify it under the terms of the GNU Lesser General Public
>> + License as published by the Free Software Foundation; either
>> + version 2.1 of the License, or (at your option) any later version.
>> +
>> + The GNU C Library is distributed in the hope that it will be useful,
>> + but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
>> + Lesser General Public License for more details.
>> +
>> + You should have received a copy of the GNU Lesser General Public
>> + License along with the GNU C Library; if not, see
>> + <http://www.gnu.org/licenses/>. */
>> +
>> +#define USE_AS_SUBMUL
>> +#include "addmul_1.S"
> I see other files with just a #include often skip the Copyright blurb.
> I'd be tempted to do the same here, but don't feel strongly about it,
> and don't know if there is a coding style in place to address it.
>
I'm still in doubt if empty implementations or simple one like this requires a full
Copyright header. Anyway, I'll be safe here and include.