From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30668 invoked by alias); 21 May 2003 15:30:57 -0000 Mailing-List: contact libc-hacker-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-hacker-owner@sources.redhat.com Received: (qmail 30631 invoked from network); 21 May 2003 15:30:56 -0000 Received: from unknown (HELO localhost.localdomain) (195.113.19.66) by sources.redhat.com with SMTP; 21 May 2003 15:30:56 -0000 Received: from sunsite.ms.mff.cuni.cz (sunsite.mff.cuni.cz [127.0.0.1]) by localhost.localdomain (8.12.8/8.12.8) with ESMTP id h4LFUrqO019422; Wed, 21 May 2003 17:30:53 +0200 Received: (from jakub@localhost) by sunsite.ms.mff.cuni.cz (8.12.8/8.12.8/Submit) id h4LFUrST019420; Wed, 21 May 2003 17:30:53 +0200 Date: Wed, 21 May 2003 15:30:00 -0000 From: Jakub Jelinek To: Ulrich Drepper , Roland McGrath Cc: Glibc hackers Subject: [PATCH] Fix ppc32 INTERNAL_SYSCALL/INLINE_SYSCALL Message-ID: <20030521153053.GF16629@sunsite.ms.mff.cuni.cz> Reply-To: Jakub Jelinek Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.4i X-SW-Source: 2003-05/txt/msg00027.txt.bz2 Hi! ppc32/NPTL doesn't build ATM since nptl/unwind.c (unwind_cleanup) uses INTERNAL_SYSCALL with string literal. Instead of adding (const char *) forever to such uses, I think it is better to change ppc sysdep.h. If an argument is a pointer type, no matter what sizeof tells it fits into a single word register. 2003-05-21 Jakub Jelinek * sysdeps/unix/sysv/linux/powerpc/powerpc32/sysdep.h (LOADARGS_0, LOADARGS_1, LOADARGS_2, LOADARGS_3, LOADARGS_4, LOADARGS_5, LOADARGS_6): Don't error if syscall argument is a string literal. --- libc/sysdeps/unix/sysv/linux/powerpc/powerpc32/sysdep.h.jj 2003-01-12 14:38:14.000000000 -0500 +++ libc/sysdeps/unix/sysv/linux/powerpc/powerpc32/sysdep.h 2003-05-21 09:22:43.000000000 -0400 @@ -117,32 +117,38 @@ # define LOADARGS_1(name, arg1) \ LOADARGS_0(name, 0); \ extern void __illegally_sized_syscall_##name##_arg1 (void); \ - if (sizeof (arg1) > 4) __illegally_sized_syscall_##name##_arg1 (); \ + if (__builtin_classify_type (arg1) != 5 && sizeof (arg1) > 4) \ + __illegally_sized_syscall_##name##_arg1 (); \ r3 = (long) (arg1) # define LOADARGS_2(name, arg1, arg2) \ LOADARGS_1(name, arg1); \ extern void __illegally_sized_syscall_##name##_arg2 (void); \ - if (sizeof (arg2) > 4) __illegally_sized_syscall_##name##_arg2 (); \ + if (__builtin_classify_type (arg2) != 5 && sizeof (arg2) > 4) \ + __illegally_sized_syscall_##name##_arg2 (); \ r4 = (long) (arg2) # define LOADARGS_3(name, arg1, arg2, arg3) \ LOADARGS_2(name, arg1, arg2); \ extern void __illegally_sized_syscall_##name##_arg3 (void); \ - if (sizeof (arg3) > 4) __illegally_sized_syscall_##name##_arg3 (); \ + if (__builtin_classify_type (arg3) != 5 && sizeof (arg3) > 4) \ + __illegally_sized_syscall_##name##_arg3 (); \ r5 = (long) (arg3) # define LOADARGS_4(name, arg1, arg2, arg3, arg4) \ LOADARGS_3(name, arg1, arg2, arg3); \ extern void __illegally_sized_syscall_##name##_arg4 (void); \ - if (sizeof (arg4) > 4) __illegally_sized_syscall_##name##_arg4 (); \ + if (__builtin_classify_type (arg4) != 5 && sizeof (arg4) > 4) \ + __illegally_sized_syscall_##name##_arg4 (); \ r6 = (long) (arg4) # define LOADARGS_5(name, arg1, arg2, arg3, arg4, arg5) \ LOADARGS_4(name, arg1, arg2, arg3, arg4); \ extern void __illegally_sized_syscall_##name##_arg5 (void); \ - if (sizeof (arg5) > 4) __illegally_sized_syscall_##name##_arg5 (); \ + if (__builtin_classify_type (arg5) != 5 && sizeof (arg5) > 4) \ + __illegally_sized_syscall_##name##_arg5 (); \ r7 = (long) (arg5) # define LOADARGS_6(name, arg1, arg2, arg3, arg4, arg5, arg6) \ LOADARGS_5(name, arg1, arg2, arg3, arg4, arg5); \ extern void __illegally_sized_syscall_##name##_arg6 (void); \ - if (sizeof (arg6) > 4) __illegally_sized_syscall_##name##_arg6 (); \ + if (__builtin_classify_type (arg6) != 5 && sizeof (arg6) > 4) \ + __illegally_sized_syscall_##name##_arg6 (); \ r8 = (long) (arg6) # define ASM_INPUT_0 "0" (r0) Jakub