Re: Bug fix for glibc broke freebsd build in REL_11_STABLE - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: Bug fix for glibc broke freebsd build in REL_11_STABLE
Date
Msg-id CAEepm=0n9=Q_BV7XgvdjEKf1_NSh-Dka3gZZYU3VpUPgUsonAA@mail.gmail.com
Whole thread Raw
In response to Re: Bug fix for glibc broke freebsd build in REL_11_STABLE  (Andrew Gierth <andrew@tao11.riddles.org.uk>)
Responses Re: Bug fix for glibc broke freebsd build in REL_11_STABLE
Re: Bug fix for glibc broke freebsd build in REL_11_STABLE
List pgsql-hackers
On Tue, Sep 4, 2018 at 9:39 AM Andrew Gierth
<andrew@tao11.riddles.org.uk> wrote:
>
> >>>>> "Andres" == Andres Freund <andres@anarazel.de> writes:
>
>  >> However, this commit broke float8 test on 32-bit FreeBSD 11 with
>  >> clang 3.8.0 compiler. Regressions.diff follows:
>
>  Andres> Does this happen with a newer clang version too?
>
> float8 test (and all other tests) passes for me on clang 3.9.1 on fbsd11
> on 32-bit ARM, and on -m32 builds on amd64.
>
> I also confirmed that without #define isinf(x) __builtin_isinf(x), on
> both 32bit and 64bit fbsd isinf() compiles as a function call, so the
> OP's proposed change would not be desirable.

I installed FreeBSD 11.2 i386 on a virtual machine.  I couldn't
reproduce the problem with either the base cc (clang 6.0.0) or clang38
(clang 3.8.1) installed via pkg.

The OP reported clang 3.8.0, so a minor version behind what I tested.

I did learn that "make check" fails in rolenames if your Unix user is
called "user".

-- 
Thomas Munro
http://www.enterprisedb.com


pgsql-hackers by date:

Previous
From: Alexander Kuzmenkov
Date:
Subject: Re: Enable using IS NOT DISTINCT FROM in hash and merge joins
Next
From: Michael Paquier
Date:
Subject: Re: Incorrect use of errcode_for_file_access in backend code