[BZ,21075] sparc - Remove unused assignment in __clone
Commit Message
The attached patch fixes:
BZ#21075 - unused assigment to %g4 in sparc/sparc{64,32}/clone.S
It is no longer needed to preserve the flags parameter to `clone' since
the commit c579f48edba88380635ab98cb612030e3ed8691e (Remove cached
PID/TID in clone).
Testing was performed successfully on sparcv9/Linux.
See the attached patch and ChangeLog entry.
Kind regards,
I.
2017-01-26 Ivo Raisr <ivo.raisr@oracle.com>
[BZ #21075]
* sysdeps/unix/sysv/linux/sparc/sparc64/clone.S (__clone): Remove
unused assignment.
* sysdeps/unix/sysv/linux/sparc/sparc32/clone.S (__clone): Likewise.
Comments
On 26/01/2017 09:02, Ivo Raisr wrote:
> The attached patch fixes:
> BZ#21075 - unused assigment to %g4 in sparc/sparc{64,32}/clone.S
>
> It is no longer needed to preserve the flags parameter to `clone' since
> the commit c579f48edba88380635ab98cb612030e3ed8691e (Remove cached
> PID/TID in clone).
>
> Testing was performed successfully on sparcv9/Linux.
>
> See the attached patch and ChangeLog entry.
>
> Kind regards,
> I.
LGTM for 2.26 since it is an optimization.
commit 79273faf7c11a76d6c9cb2b48c700eda772928e6
Author: Ivo Raisr <ivo.raisr@oracle.com>
Date: Tue Jan 24 10:31:22 2017 +0100
sparc: Remove unused assignment in __clone [BZ #21075]
ChangeLog:
[BZ #21075]
* sysdeps/unix/sysv/linux/sparc/sparc64/clone.S (__clone): Remove
unused assignment.
* sysdeps/unix/sysv/linux/sparc/sparc32/clone.S (__clone): Likewise.
@@ -48,7 +48,6 @@ ENTRY (__clone)
expects. */
sub %o1, 96, %o1
mov %i3, %g3
- mov %i2, %g4
/* ptid */
mov %i4,%o2
@@ -51,7 +51,6 @@ ENTRY (__clone)
expects. Also, subtract STACK_BIAS. */
sub %i1, 192 + 0x7ff, %o1
mov %i3, %g3
- mov %i2, %g4
mov %i4,%o2 /* PTID */
mov %i5,%o3 /* TLS */