Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 20 Sep 2018 11:54:02 -0400
From:      Mark Johnston <markj@freebsd.org>
To:        Matt Macy <mmacy@freebsd.org>, jmd@freebsd.org
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r336299 - in head: include lib/msun lib/msun/ld128 lib/msun/ld80 lib/msun/man lib/msun/src
Message-ID:  <20180920155402.GF99168@raichu>
In-Reply-To: <201807150023.w6F0NBx1065422@repo.freebsd.org>
References:  <201807150023.w6F0NBx1065422@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Jul 15, 2018 at 12:23:11AM +0000, Matt Macy wrote:
> Author: mmacy
> Date: Sun Jul 15 00:23:10 2018
> New Revision: 336299
> URL: https://svnweb.freebsd.org/changeset/base/336299
> 
> Log:
>   msun: add ld80/ld128 powl, cpow, cpowf, cpowl from openbsd
>   
>   This corresponds to the latest status (hasn't changed in 9+
>   years) from openbsd of ld80/ld128 powl, and source cpowf, cpow,
>   cpowl (the complex power functions for float complex, double
>   complex, and long double complex) which are required for C99
>   compliance and were missing from FreeBSD. Also required for
>   some numerical codes using complex numbered Hamiltonians.
>   
>   Thanks to jhb for tracking down the issue with making
>   weak_reference compile on powerpc.
>   
>   When asked to review, bde said "I don't like it" - but
>   provided no actionable feedback or superior implementations.
>   
>   Discussed with: jhb
>   Submitted by: jmd
>   Differential Revision: https://reviews.freebsd.org/D15919

This seems to have broken the gcc build:
https://ci.freebsd.org/job/FreeBSD-head-amd64-gcc/

/workspace/src/lib/msun/ld80/e_powl.c:275:1: error: floating constant exceeds range of 'long double' [-Werror=overflow]
  if( y >= LDBL_MAX )
  ^~



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180920155402.GF99168>