From: Tristan Gingold <gingold@adacore.com>
To: H.J. Lu <hjl.tools@gmail.com>
Cc: Matthias Klose <doko@ubuntu.com>,
GCC Patches <gcc-patches@gcc.gnu.org>,
binutils <binutils@sourceware.org>
Subject: Re: Review of --enable-gold=both patch
Date: Wed, 24 Nov 2010 13:25:00 -0000 [thread overview]
Message-ID: <A3D84A54-6AFA-4017-88FB-1E3BB8D0B80B@adacore.com> (raw)
In-Reply-To: <AANLkTimzc=xo=VN0u7h5Oe1vR-zFMJmH1soL90HOvOmZ@mail.gmail.com>
On Nov 24, 2010, at 2:11 PM, H.J. Lu wrote:
> On Wed, Nov 24, 2010 at 2:00 AM, Tristan Gingold <gingold@adacore.com> wrote:
>>
>> On Nov 23, 2010, at 7:02 PM, H.J. Lu wrote:
>>
>>>
>>> This change:
>>>
>>> case "${ENABLE_LD}" in
>>> default)
>>> if test x${default_ld} != xgold; then
>>> AC_MSG_ERROR([either gold or ld can be the default ld])
>>> fi
>>> ;;
>>>
>>> is wrong. At this time, default_ld is set to gold only by --enable-gold=default
>>> What we should test is
>>>
>>> if test x${default_ld} != x; then
>>>
>>> which checks that default_ld isn't set. Also comments are wrong:
>>>
>>> # Handle --enable-gold, --enable-ld.
>>> # --disable-gold [--enable-ld]
>>> # Build only ld. Default option.
>>> # --enable-gold[=default] [--enable-ld]
>>> # Build both gold and ld. Install gold as "ld.gold" and "ld",
>>> # install ld as "ld.bfd".
>>>
>>> --enable-gold only enables gold, not make it default unless
>>> ld is disabled.
>>>
>>> # --enable-gold[=default] --disable-ld
>>> # Build only gold, which is then installed as both "ld.gold" and "ld".
>>> # --enable-gold --enable-ld=default
>>> # Build both gold (installed as "gold") and ld (installed as "ld").
>>>
>>> gold is installed as ld.gold. ld is also installed ld.bfd.
>>>
>>> # In other words, ld is default
>>> # --enable-gold=default --enable-ld=default
>>> # Error.
>>>
>>> I am checking in this patch as an obvious fix.
>>
>> Looks like this hasn't yet been committed to binutils.
>>
>
> I don't know what happened to cvs archive. But it is in:
>
> http://sourceware.org/git/?p=binutils.git;a=commit;h=e1cffffd802eb3d50c4f3e0718d1f194c58d8dc0
Indeed.
Tristan.
next prev parent reply other threads:[~2010-11-24 13:25 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4C27E695.9070802@codesourcery.com>
[not found] ` <4C286200.30402@gnu.org>
[not found] ` <Pine.LNX.4.64.1006281009290.17565@digraph.polyomino.org.uk>
[not found] ` <4C2886C7.4040408@gnu.org>
[not found] ` <4C28CA2A.7060703@codesourcery.com>
[not found] ` <4C28CE9C.9030809@gnu.org>
[not found] ` <4C28D9F1.6030309@codesourcery.com>
2010-10-27 14:04 ` Matthias Klose
2010-10-29 9:21 ` Nick Clifton
2010-10-29 14:55 ` Matthias Klose
2010-10-29 15:00 ` Paolo Bonzini
2010-10-29 15:12 ` Paolo Bonzini
2010-10-29 15:41 ` Matthias Klose
2010-10-29 15:42 ` Paolo Bonzini
2010-11-01 20:21 ` Ralf Wildenhues
2010-11-11 0:09 ` Matthias Klose
2010-11-11 8:37 ` Paolo Bonzini
2010-11-12 18:45 ` Matthias Klose
2010-11-12 19:36 ` Paolo Bonzini
[not found] ` <4CDD8B19.3040909__3908.7257617668$1289587526$gmane$org@ubuntu.com>
2010-11-12 20:37 ` Andreas Schwab
2010-11-13 20:04 ` Matthias Klose
2010-11-13 20:48 ` Paolo Bonzini
[not found] ` <4CDEEEEB.2000905__41684.8152097829$1289678668$gmane$org@ubuntu.com>
2010-11-14 7:25 ` Ryan Hill
2010-11-23 12:24 ` Nick Clifton
2010-11-23 13:27 ` Matthias Klose
2010-11-23 13:31 ` Tristan Gingold
2010-11-23 17:09 ` H.J. Lu
2010-11-23 18:02 ` H.J. Lu
2010-11-24 10:00 ` Tristan Gingold
2010-11-24 13:11 ` H.J. Lu
2010-11-24 13:25 ` Tristan Gingold [this message]
2010-11-23 18:13 ` H.J. Lu
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=A3D84A54-6AFA-4017-88FB-1E3BB8D0B80B@adacore.com \
--to=gingold@adacore.com \
--cc=binutils@sourceware.org \
--cc=doko@ubuntu.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=hjl.tools@gmail.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).