[PATCH] PPC make aligned_restore_vmx a local symbol.

Steven Munroe munroesj@linux.vnet.ibm.com
Thu Jan 8 17:14:00 GMT 2009


__longjmp exports a symbol "aligned_restore_vmx" which should only be
local. This sometimes confuses Oprofile and related tools. It is an
oversight that this label is not wrapped with the L() macro.

The attached patch fixes this.


-------------- next part --------------
2009-01-06  Steven Munroe  <sjmunroe@us.ibm.com>

	* sysdeps/powerpc/powerpc32/fpu/__longjmp-common.S (setcontext):
	Make aligned_restore_vmx a local symbol.
	* sysdeps/powerpc/powerpc64/__longjmp-common.S (setcontext):
	Make aligned_restore_vmx a local symbol.

diff -urN libc29-cvs-20090106/sysdeps/powerpc/powerpc32/fpu/__longjmp-common.S libc29/sysdeps/powerpc/powerpc32/fpu/__longjmp-common.S
--- libc29-cvs-20090106/sysdeps/powerpc/powerpc32/fpu/__longjmp-common.S	2008-11-16 19:34:22.000000000 -0600
+++ libc29/sysdeps/powerpc/powerpc32/fpu/__longjmp-common.S	2009-01-07 15:13:27.000000000 -0600
@@ -64,7 +64,7 @@
 	andi.	r6,r5,0xf
 	lwz	r0,((JB_VRSAVE)*4)(3)
 	mtspr	VRSAVE,r0
-	beq+	aligned_restore_vmx
+	beq+	L(aligned_restore_vmx)
 	addi    r6,r5,16
 	lvsl	v0,0,r5
 	lvx	v1,0,r5
@@ -88,7 +88,7 @@
 	lvx	v1,0,r5
 	vperm   v31,v31,v1,v0
 	b       L(no_vmx)
-aligned_restore_vmx:
+L(aligned_restore_vmx):
 	addi	r6,r5,16
 	lvx	v20,0,r5
 	addi	r5,r5,32
diff -urN libc29-cvs-20090106/sysdeps/powerpc/powerpc64/__longjmp-common.S libc29/sysdeps/powerpc/powerpc64/__longjmp-common.S
--- libc29-cvs-20090106/sysdeps/powerpc/powerpc64/__longjmp-common.S	2006-09-22 01:06:43.000000000 -0500
+++ libc29/sysdeps/powerpc/powerpc64/__longjmp-common.S	2009-01-07 15:13:56.000000000 -0600
@@ -58,7 +58,7 @@
 	andi.	r6,r5,0xf
 	lwz	r0,((JB_VRSAVE)*8)(3)
 	mtspr	VRSAVE,r0
-	beq+	aligned_restore_vmx
+	beq+	L(aligned_restore_vmx)
 	addi    r6,r5,16
 	lvsl	v0,0,r5
 	lvx	v1,0,r5
@@ -82,7 +82,7 @@
 	lvx	v1,0,r5
 	vperm   v31,v31,v1,v0
 	b       L(no_vmx)
-aligned_restore_vmx:
+L(aligned_restore_vmx):
 	addi	r6,r5,16
 	lvx	v20,0,r5
 	addi	r5,r5,32


More information about the Libc-alpha mailing list