public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: "H. J. Lu" <hjl@lucon.org>
To: binutils@sources.redhat.com
Subject: PATCH: ld/1485: --enable-targets=all doesn't work for 64bit target
Date: Thu, 25 May 2006 03:19:00 -0000	[thread overview]
Message-ID: <20060524172325.GA4375@lucon.org> (raw)

This patch checks 64bit bfd properly so that --enable-targets=all will
work for 64bit target and host.


H.J.
----
2006-05-24  H.J. Lu  <hongjiu.lu@intel.com>

	PR ld/1485
	* configure.in: Use ${srcdir}/../bfd/config.bfd to check 64bit
	bfd.  Support 64bit host for --enable-targets=all.
	* configure: Regenerated.

--- ld/configure.in.64	2006-04-14 14:44:46.000000000 -0700
+++ ld/configure.in	2006-05-24 10:16:18.000000000 -0700
@@ -169,6 +169,11 @@ do
       EMUL=$targ_emul
     fi
 
+    . ${srcdir}/../bfd/config.bfd
+    case "${targ_defvec}-${targ_selvecs}" in
+    *elf64*) want64=true;;
+    esac
+
     for i in $targ_emul $targ_extra_emuls $targ_extra_libpath; do
 	case " $all_emuls " in
 	*" e${i}.o "*) ;;
@@ -210,10 +215,13 @@ AC_SUBST(EMUL)
 TDIRS=tdirs
 AC_SUBST_FILE(TDIRS)
 
-dnl FIXME: We will build a 64 bit BFD for a 64 bit host or a 64 bit
-dnl target, and in those cases we should also build the 64 bit
-dnl emulations.
 if test x${all_targets} = xtrue; then
+  if test x${want64} = xfalse; then
+    AC_CHECK_SIZEOF(long)
+    if test "x${ac_cv_sizeof_long}" = "x8"; then
+      want64=true
+    fi
+  fi
   if test x${want64} = xtrue; then
     EMULATION_OFILES='$(ALL_EMULATIONS) $(ALL_64_EMULATIONS)'
   else

             reply	other threads:[~2006-05-24 17:23 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-25  3:19 H. J. Lu [this message]
2006-05-25  3:48 ` Alan Modra
2006-05-25  4:20   ` H. J. Lu
2006-05-25  5:01     ` Alan Modra
2006-05-25  5:51       ` Should sh-*-* enable 64bit targets? H. J. Lu
2006-05-25  5:55         ` Alan Modra
2006-05-25 14:45           ` H. J. Lu
2006-05-25 14:56             ` Alan Modra

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=20060524172325.GA4375@lucon.org \
    --to=hjl@lucon.org \
    --cc=binutils@sources.redhat.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).