This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: Patch: Add __restrict to string.h
- From: Peter Rosin <peda at lysator dot liu dot se>
- To: Joel Sherrill <joel dot sherrill at oarcorp dot com>
- Cc: "newlib at sourceware dot org" <newlib at sourceware dot org>
- Date: Wed, 14 Aug 2013 15:43:41 +0200
- Subject: Re: Patch: Add __restrict to string.h
- References: <51E829F4 dot 60706 at oarcorp dot com> <520B775A dot 1020003 at lysator dot liu dot se> <520B7F17 dot 7080302 at lysator dot liu dot se> <520B83DF dot 2030005 at oarcorp dot com>
On 2013-08-14 15:19, Joel Sherrill wrote:
> On 8/14/2013 7:59 AM, Peter Rosin wrote:
>> On 2013-08-14 14:26, Peter Rosin wrote:
>>> On 2013-07-18 19:46, Joel Sherrill wrote:
>>>> Index: libc/machine/xscale/memcpy.c
>>>> ===================================================================
>>>> RCS file: /cvs/src/src/newlib/libc/machine/xscale/memcpy.c,v
>>>> retrieving revision 1.2
>>>> diff -u -r1.2 memcpy.c
>>>> --- libc/machine/xscale/memcpy.c 20 Apr 2002 00:29:51 -0000 1.2
>>>> +++ libc/machine/xscale/memcpy.c 18 Jul 2013 17:44:19 -0000
>>>> @@ -8,7 +8,7 @@
>>>> #include "xscale.h"
>>>> void *
>>>> -memcpy (void *dst0, const void *src0, size_t len)
>>>> +memcpy (void *_restrict dst0, const void *_restrict src0, size_t len)
>>>> {
>>>> int dummy;
>>>> asm volatile (
>>> Missing an underscore? I.e. __restrict instead of _restrict?
>> That was fixed in v3. Sorry for not finishing reading...
> No problem. Is it correct in CVS? That's what matters.
Since I caused this fuzz I double-checked, and it looks good in CVS.
Cheers,
Peter