[roland/nptl-sh] SH: Consolidate NPTL/non versions of vfork

Message ID 20140531.110353.293156792.kkojima@rr.iij4u.or.jp
State Committed
Headers

Commit Message

Kaz Kojima May 31, 2014, 2:03 a.m. UTC
  Roland McGrath <roland@hack.frob.com> wrote:
> I've made that fix on the branch.  Please let me know if it works now.
> (Or feel free to just commit it yourself if you prefer.)

I've got

/tmp/ccAPJUC8.s: Assembler messages:
/tmp/ccAPJUC8.s: Error: .size expression for __vfork does not evaluate to a constant

when compiling.  From alpha/vfork.S, the patch below may fix
the issue.  With it in origin/roland/nptl-sh branch, it looks
vfork objects are in good shape.

Regards,
	kaz
--
  

Comments

Roland McGrath June 2, 2014, 9:30 p.m. UTC | #1
I applied the fix and committed the change.  I hope I didn't break SH. :-)
  

Patch

diff --git a/sysdeps/unix/sysv/linux/sh/vfork.S b/sysdeps/unix/sysv/linux/sh/vfork.S
index 157a38c..76326f5 100644
--- a/sysdeps/unix/sysv/linux/sh/vfork.S
+++ b/sysdeps/unix/sysv/linux/sh/vfork.S
@@ -64,10 +64,7 @@  ENTRY (__libc_vfork)
 .L2:
 	.word	PID - TLS_PRE_TCB_SIZE
 	.align	2
-PSEUDO_END (__vfork)
-libc_hidden_def (__vfork)
-
-weak_alias (__vfork, vfork)
+PSEUDO_END (__libc_vfork)
 
 #if !NOT_IN_libc
 weak_alias (__libc_vfork, vfork)