Re: [PATCH, rs6000] Modify libgcc's float128 IFUNC resolver functions to use __builtin_cpu_supports()

2017-07-20 Thread Peter Bergner
On 7/10/17 2:52 PM, Peter Bergner wrote: > On 7/10/17 9:48 AM, Segher Boessenkool wrote: >> On Fri, Jul 07, 2017 at 07:14:25PM -0500, Peter Bergner wrote: >>> On 7/7/17 4:13 PM, Peter Bergner wrote: On 7/7/17 10:18 AM, Segher Boessenkool wrote: > On Thu, Jul 06, 2017 at 04:21:48PM -0500,

Re: [PATCH, rs6000] Modify libgcc's float128 IFUNC resolver functions to use __builtin_cpu_supports()

2017-07-10 Thread Peter Bergner
On 7/10/17 9:48 AM, Segher Boessenkool wrote: > On Fri, Jul 07, 2017 at 07:14:25PM -0500, Peter Bergner wrote: >> On 7/7/17 4:13 PM, Peter Bergner wrote: >>> On 7/7/17 10:18 AM, Segher Boessenkool wrote: On Thu, Jul 06, 2017 at 04:21:48PM -0500, Peter Bergner wrote: > *

Re: [PATCH, rs6000] Modify libgcc's float128 IFUNC resolver functions to use __builtin_cpu_supports()

2017-07-10 Thread Segher Boessenkool
On Fri, Jul 07, 2017 at 07:14:25PM -0500, Peter Bergner wrote: > On 7/7/17 4:13 PM, Peter Bergner wrote: > > On 7/7/17 10:18 AM, Segher Boessenkool wrote: > >> On Thu, Jul 06, 2017 at 04:21:48PM -0500, Peter Bergner wrote: > >>> * config/rs6000/float128-ifunc.c: Don't include auxv.h. > >>>

Re: [PATCH, rs6000] Modify libgcc's float128 IFUNC resolver functions to use __builtin_cpu_supports()

2017-07-07 Thread Peter Bergner
On 7/7/17 4:13 PM, Peter Bergner wrote: > On 7/7/17 10:18 AM, Segher Boessenkool wrote: >> On Thu, Jul 06, 2017 at 04:21:48PM -0500, Peter Bergner wrote: >>> * config/rs6000/float128-ifunc.c: Don't include auxv.h. >>> (have_ieee_hw_p): Delete function. >>> (SW_OR_HW) Use

Re: [PATCH, rs6000] Modify libgcc's float128 IFUNC resolver functions to use __builtin_cpu_supports()

2017-07-07 Thread Peter Bergner
On 7/7/17 10:18 AM, Segher Boessenkool wrote: > On Thu, Jul 06, 2017 at 04:21:48PM -0500, Peter Bergner wrote: >> * config/rs6000/float128-ifunc.c: Don't include auxv.h. >> (have_ieee_hw_p): Delete function. >> (SW_OR_HW) Use __builtin_cpu_supports(). > > Okay for trunk. Thanks!

Re: [PATCH, rs6000] Modify libgcc's float128 IFUNC resolver functions to use __builtin_cpu_supports()

2017-07-07 Thread Segher Boessenkool
On Thu, Jul 06, 2017 at 04:21:48PM -0500, Peter Bergner wrote: > * config/rs6000/float128-ifunc.c: Don't include auxv.h. > (have_ieee_hw_p): Delete function. > (SW_OR_HW) Use __builtin_cpu_supports(). Okay for trunk. Thanks! Segher

Re: [PATCH, rs6000] Modify libgcc's float128 IFUNC resolver functions to use __builtin_cpu_supports()

2017-07-07 Thread Florian Weimer
On 07/06/2017 11:21 PM, Peter Bergner wrote: > I will note that this patch causes issues in some tests in the GLIBC > testsiute, > which Tulio is working on fixing (it's a GLIBC issue, not a GCC issue), so if > this patch is "ok", I plan on holding off on committing this, until the GLIBC > fix is

[PATCH, rs6000] Modify libgcc's float128 IFUNC resolver functions to use __builtin_cpu_supports()

2017-07-06 Thread Peter Bergner
Usage of getauxval() within the float128 libgcc IFUNC resolver functions is causing problems: https://sourceware.org/bugzilla/show_bug.cgi?id=21707 Alan describes why we can't have relocations in IFUNC resolver functions here: https://gcc.gnu.org/PR81193 With the addition of