public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Joseph Myers <joseph@codesourcery.com>
To: Florian Weimer <fweimer@redhat.com>
Cc: Carlos O'Donell <carlos@redhat.com>,
	Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>,
	<libc-alpha@sourceware.org>, <bergner@vnet.ibm.com>,
	<gftg@linux.vnet.ibm.com>
Subject: Re: [RFC] Fix float128 IFUNC relocations on ppc64le [BZ #21707]
Date: Wed, 05 Jul 2017 20:12:00 -0000	[thread overview]
Message-ID: <alpine.DEB.2.20.1707052002550.14914@digraph.polyomino.org.uk> (raw)
In-Reply-To: <73e06e98-15f5-d4f9-530b-4a6c35c76ff8@redhat.com>

On Wed, 5 Jul 2017, Florian Weimer wrote:

> On 07/05/2017 09:25 PM, Carlos O'Donell wrote:
> >>> +# libgcc requires __tcb_parse_hwcap_and_convert_at_platform when built with
> >>> +# a binary128 type.  That symbol is provided by the loader on dynamically
> >>> +# linked executables, forcing to link the loader after libgcc link.
> >>> +f128-loader-link = $(as-needed) $(elf-objpfx)ld.so $(no-as-needed)
> >> Why doesn't the regular linker invocation take care of this?  Will user
> >> applications run into the same issue?
> > We don't use the regular linker invocation in glibc builds? We have to emulate
> > what the actual toolchain would do in this case.
> 
> But then we should do this everywhere, and not just for the few tests
> that actually need it, right?  I assume that's also what the official
> linker script does.

The installed compiler does -lgcc -lc -lgcc (roughly; -lgcc may actually 
use -lgcc_s or -lgcc_eh, and -lc may vary for e.g. profiling).  -lc is a 
linker script doing e.g. GROUP ( /lib64/libc.so.6 
/usr/lib64/libc_nonshared.a  AS_NEEDED ( /lib64/ld-linux-x86-64.so.2 ) ).  
The shared libgcc is also a linker script on some platforms.

The glibc build uses libc.so --as-needed ld.so --no-as-needed -lgcc 
(again, roughly) (but libc.a -lgcc libc.a -lgcc in the static build case).

I suppose the suggestion is that we should both use -lgcc before linking 
with libc, not just after (this being the issue involved in the present 
case), and use -Wl,--start-group -Wl,--end-group around libc.so and ld.so, 
both to be more similar to how things work with the installed compiler?  
Which makes sense, but is also rather risky for this development stage.

-- 
Joseph S. Myers
joseph@codesourcery.com

  reply	other threads:[~2017-07-05 20:12 UTC|newest]

Thread overview: 83+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-23 12:33 [PATCH v3 0/7] Enable float128 on powerpc64le Gabriel F. T. Gomes
2017-06-23 12:33 ` [PATCH v3 3/7] Add libio-mtsafe flags to the build of strfromf128 Gabriel F. T. Gomes
2017-06-23 12:59   ` Joseph Myers
2017-06-23 12:33 ` [PATCH v3 1/7] Include libc-header-start.h in include/float.h Gabriel F. T. Gomes
2017-06-23 12:34 ` [PATCH v3 6/7] powerpc64le: Check for compiler features for float128 Gabriel F. T. Gomes
2017-06-23 16:37   ` Joseph Myers
2017-06-23 20:47     ` Steven Munroe
2017-06-23 12:34 ` [PATCH v3 4/7] Update string to float128 functions to use bits/types/locale_t.h Gabriel F. T. Gomes
2017-06-23 12:57   ` Gabriel F. T. Gomes
2017-06-23 12:34 ` [PATCH v3 2/7] Prepare the manual to display math errors for float128 functions Gabriel F. T. Gomes
2017-06-23 12:34 ` [PATCH v3 5/7] Document _FloatN and _FloatNx versions of math functions Gabriel F. T. Gomes
2017-06-23 13:11   ` Joseph Myers
2017-06-23 12:34 ` [PATCH v3 7/7] powerpc64le: Enable float128 Gabriel F. T. Gomes
2017-06-23 13:18   ` Joseph Myers
2017-06-26 17:11   ` Tulio Magno Quites Machado Filho
2017-06-26 23:02     ` Joseph Myers
2017-06-27  3:29       ` Alan Modra
2017-06-27  7:05       ` Florian Weimer
2017-06-27 15:00         ` Peter Bergner
2017-06-27 18:42           ` Florian Weimer
2017-06-27 18:51             ` Peter Bergner
2017-06-27 19:17               ` Florian Weimer
2017-06-27 19:33                 ` Carlos Eduardo Seo
2017-06-27 19:34                 ` Peter Bergner
2017-06-27 21:41                   ` Florian Weimer
2017-06-28 16:30                     ` Szabolcs Nagy
2017-06-28 17:08                       ` Peter Bergner
2017-06-28 17:15                         ` Szabolcs Nagy
2017-07-04  6:38         ` ppc64le: getauxval call from IFUNC resolver Florian Weimer
2017-07-05 17:25           ` [RFC] Fix float128 IFUNC relocations on ppc64le [BZ #21707] Tulio Magno Quites Machado Filho
2017-07-05 19:19             ` Florian Weimer
2017-07-05 19:25               ` Carlos O'Donell
2017-07-05 19:44                 ` Florian Weimer
2017-07-05 20:12                   ` Joseph Myers [this message]
2017-07-06  6:19                     ` Florian Weimer
2017-07-06 14:04                       ` Carlos O'Donell
2017-07-07 10:08                         ` Florian Weimer
2017-07-08 18:31                           ` [PATCHv2] " Tulio Magno Quites Machado Filho
2017-07-08 19:17                             ` Florian Weimer
2017-07-08 23:18                               ` Tulio Magno Quites Machado Filho
2017-07-08 19:30                             ` H.J. Lu
2017-07-08 23:12                               ` Tulio Magno Quites Machado Filho
2017-07-08 23:59                                 ` H.J. Lu
2017-07-09 14:59                                   ` [PATCH, 2.27] x86-64: Use IFUNC memcpy and mempcpy in libc.a H.J. Lu
2017-08-02 15:53                                     ` H.J. Lu
2017-08-02 18:27                                       ` Florian Weimer
2017-08-02 18:36                                         ` H.J. Lu
2017-08-02 18:51                                           ` Florian Weimer
2017-08-04 13:18                                             ` H.J. Lu
2017-07-09 16:28                             ` [PATCHv2] Fix float128 IFUNC relocations on ppc64le [BZ #21707] Nix
2017-07-10 15:28                               ` Tulio Magno Quites Machado Filho
2017-07-10 15:37                                 ` Nix
2017-07-12 13:45                                   ` Nix
2017-07-10 10:09                             ` Joseph Myers
2017-07-10 18:02                               ` Tulio Magno Quites Machado Filho
2017-07-10 15:04                             ` Carlos O'Donell
2017-07-12 18:59                               ` [PATCHv3] powerpc: Fix float128 IFUNC relocations " Tulio Magno Quites Machado Filho
2017-07-12 19:42                                 ` H.J. Lu
2017-07-12 20:58                                 ` Carlos O'Donell
2017-07-13 14:29                                   ` Tulio Magno Quites Machado Filho
2017-07-13 14:33                                     ` Carlos O'Donell
2017-07-17 14:06                                       ` [PATCHv4] " Tulio Magno Quites Machado Filho
2017-07-17 15:46                                         ` Carlos O'Donell
2017-07-17 16:27                                           ` Tulio Magno Quites Machado Filho
2017-07-05 19:21             ` [RFC] Fix float128 IFUNC relocations on ppc64le " Carlos O'Donell
2017-07-08 18:46               ` Tulio Magno Quites Machado Filho
2017-07-05 19:40             ` Gabriel F. T. Gomes
2017-07-05 20:32               ` Tulio Magno Quites Machado Filho
2017-07-07 22:19           ` ppc64le: getauxval call from IFUNC resolver Peter Bergner
2017-07-10 19:59             ` Peter Bergner
2017-07-20 14:48               ` Peter Bergner
2017-06-27 17:46       ` [PATCH v3 7/7] powerpc64le: Enable float128 Gabriel F. T. Gomes
2017-06-27  7:20   ` Andreas Schwab
2017-06-27  9:24   ` Andreas Schwab
2017-06-27 13:15   ` Andreas Schwab
2017-06-28  0:54     ` [PATCH] powerpc64le: Iterate over all object suffixes when appending -mfloat128 Gabriel F. T. Gomes
2017-06-28  7:19       ` Andreas Schwab
2017-06-28 11:58         ` Gabriel F. T. Gomes
2017-06-28  7:22       ` Andreas Schwab
2017-07-10 10:22       ` Joseph Myers
2017-07-11 14:07         ` Tulio Magno Quites Machado Filho
2017-07-11 17:21           ` Gabriel F. T. Gomes
2017-06-23 12:59 ` [PATCH v3 0/7] Enable float128 on powerpc64le Joseph Myers

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=alpine.DEB.2.20.1707052002550.14914@digraph.polyomino.org.uk \
    --to=joseph@codesourcery.com \
    --cc=bergner@vnet.ibm.com \
    --cc=carlos@redhat.com \
    --cc=fweimer@redhat.com \
    --cc=gftg@linux.vnet.ibm.com \
    --cc=libc-alpha@sourceware.org \
    --cc=tuliom@linux.vnet.ibm.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).