From: "Maciej W. Rozycki" <macro@linux-mips.org>
To: David Daney <ddaney@avtrex.com>
Cc: Richard Sandiford <richard@codesourcery.com>,
GCJ-patches <java-patches@gcc.gnu.org>,
GCC Patches <gcc-patches@gcc.gnu.org>
Subject: Re: [Patch] libffi: Add MIPS64 support.
Date: Mon, 13 Aug 2007 17:56:00 -0000 [thread overview]
Message-ID: <Pine.LNX.4.64N.0708131549050.2722@blysk.ds.pg.gda.pl> (raw)
In-Reply-To: <46BD4BDE.8030407@avtrex.com>
David,
While integrating into 4.1.2 I had a brief look over your patch again
over the weekend and I have got two minor points:
1. You use "unsigned" rather than "unsigned int", which makes your code
look a little bit too K&Rish.
2. Given the coverage FFI_MIPS_N32 should probably be renamed to
FFI_MIPS_NEWABI as it is now somewhat confusing.
And last but not least, with your changes there is now no difference
between Linux and IRIX anymore, so it may be worth unifying the MIPS
target like the others; it seems to be the only oddball.
2007-08-13 Maciej W. Rozycki <macro@linux-mips.org>
* Makefile.am: Unify MIPS_IRIX and MIPS_LINUX into MIPS.
* configure.ac: Likewise.
* Makefile.in: Regenerate.
* configure: Regenerate.
OK to apply?
Maciej
gcc-4.3.0-20070813-mips64-linux-libffi.patch
diff -up --recursive --new-file gcc-4.3.0-20070813.macro/libffi/Makefile.am gcc-4.3.0-20070813/libffi/Makefile.am
--- gcc-4.3.0-20070813.macro/libffi/Makefile.am 2007-08-11 06:55:52.000000000 +0000
+++ gcc-4.3.0-20070813/libffi/Makefile.am 2007-08-13 16:52:18.000000000 +0000
@@ -84,10 +84,7 @@ libffi_la_SOURCES = src/debug.c src/prep
nodist_libffi_la_SOURCES =
-if MIPS_IRIX
-nodist_libffi_la_SOURCES += src/mips/ffi.c src/mips/o32.S src/mips/n32.S
-endif
-if MIPS_LINUX
+if MIPS
nodist_libffi_la_SOURCES += src/mips/ffi.c src/mips/o32.S src/mips/n32.S
endif
if X86
diff -up --recursive --new-file gcc-4.3.0-20070813.macro/libffi/configure.ac gcc-4.3.0-20070813/libffi/configure.ac
--- gcc-4.3.0-20070813.macro/libffi/configure.ac 2007-08-11 06:55:52.000000000 +0000
+++ gcc-4.3.0-20070813/libffi/configure.ac 2007-08-13 16:53:00.000000000 +0000
@@ -95,10 +95,10 @@ case "$host" in
;;
mips-sgi-irix5.* | mips-sgi-irix6.*)
- TARGET=MIPS_IRIX; TARGETDIR=mips
+ TARGET=MIPS; TARGETDIR=mips
;;
mips*-*-linux*)
- TARGET=MIPS_LINUX; TARGETDIR=mips
+ TARGET=MIPS; TARGETDIR=mips
;;
powerpc*-*-linux* | powerpc-*-sysv*)
@@ -151,8 +151,7 @@ if test $TARGETDIR = unknown; then
AC_MSG_ERROR(["libffi has not been ported to $host."])
fi
-AM_CONDITIONAL(MIPS_IRIX, test x$TARGET = xMIPS_IRIX)
-AM_CONDITIONAL(MIPS_LINUX, test x$TARGET = xMIPS_LINUX)
+AM_CONDITIONAL(MIPS, test x$TARGET = xMIPS)
AM_CONDITIONAL(SPARC, test x$TARGET = xSPARC)
AM_CONDITIONAL(X86, test x$TARGET = xX86)
AM_CONDITIONAL(X86_WIN32, test x$TARGET = xX86_WIN32)
@@ -176,11 +175,6 @@ AM_CONDITIONAL(PA_LINUX, test x$TARGET =
AM_CONDITIONAL(PA_HPUX, test x$TARGET = xPA_HPUX)
AM_CONDITIONAL(PA64_HPUX, test x$TARGET = xPA64_HPUX)
-case x$TARGET in
- xMIPS*) TARGET=MIPS ;;
- *) ;;
-esac
-
AC_HEADER_STDC
AC_CHECK_FUNCS(memcpy)
AC_FUNC_ALLOCA
next prev parent reply other threads:[~2007-08-13 17:56 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-09 5:01 David Daney
2007-08-09 9:58 ` Richard Sandiford
2007-08-09 15:50 ` David Daney
2007-08-09 15:59 ` Richard Sandiford
2007-08-09 17:28 ` Maciej W. Rozycki
2007-08-09 17:31 ` David Daney
2007-08-10 10:58 ` Maciej W. Rozycki
2007-08-10 15:26 ` David Daney
2007-08-10 17:22 ` Maciej W. Rozycki
2007-08-11 5:43 ` David Daney
2007-08-13 17:56 ` Maciej W. Rozycki [this message]
2007-08-13 18:19 ` David Daney
2007-08-13 20:51 ` Tom Tromey
2007-08-14 11:53 ` Maciej W. Rozycki
2007-08-15 6:40 ` David Daney
2007-08-17 20:23 ` Tom Tromey
2007-09-03 18:09 ` Maciej W. Rozycki
2007-08-13 18:38 ` Andreas Schwab
2007-08-10 15:49 ` David Daney
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Pine.LNX.4.64N.0708131549050.2722@blysk.ds.pg.gda.pl \
--to=macro@linux-mips.org \
--cc=ddaney@avtrex.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=java-patches@gcc.gnu.org \
--cc=richard@codesourcery.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).