Date: Thu, 29 Jun 2017 10:33:09 -0700 From: Mark Millard <markmi@dsl-only.net> To: Dimitry Andric <dim@FreeBSD.org> Cc: Justin Hibbits <jhibbits@FreeBSD.org>, FreeBSD Toolchain <freebsd-toolchain@freebsd.org>, FreeBSD PowerPC ML <freebsd-ppc@freebsd.org>, FreeBSD Current <freebsd-current@freebsd.org> Subject: Re: head -r320458 (e.g.) amd64 -> powerpc (32-bit) cross-buildworld fails for time libc++ static_asserts and overflow/underflow of long long (system clang/clang++ 4 based build) Message-ID: <ACD8EAE8-9D33-4E0D-B3AC-E7953544DD1F@dsl-only.net> In-Reply-To: <55A1694D-BC40-49AE-BEF7-CEE502126E96@FreeBSD.org> References: <D8AB364D-9E4B-470A-A858-842E73530C3B@dsl-only.net> <5B12D9A6-C9A2-4B0D-B32D-D04D7DB1E3BC@dsl-only.net> <AFCE7DA0-E8D2-4FC7-9E2F-78011E54F1A0@dsl-only.net> <55A1694D-BC40-49AE-BEF7-CEE502126E96@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[Good news from the llvm side of things. . .]
On 2017-Jun-29, at 3:47 AM, Dimitry Andric <dim at FreeBSD.org> wrote:
> On 29 Jun 2017, at 12:04, Mark Millard <markmi at dsl-only.net> wrote:
>>=20
>> [The libc++ code in question appears to not be ready for
>> 32-bit contexts with 64 bit times. Disable
>> experimental/filesystem for now? I've submitted
>> llvm bugzilla 33638 for the issue and have
>> added it to llvm's 25780, the FreeBSD META for
>> clang.]
>=20
> Yes, this also broke earlier on arm and mips, which is why there is =
the
> following in lib/Makefile:
>=20
> .if ${MACHINE_CPUARCH} !=3D "arm" && ${MACHINE_CPUARCH} !=3D "mips"
> _libcplusplus+=3D libc++experimental
> .endif
>=20
> I haven't yet found the time to address this issue. Upstream should
> already be aware of it, though.
>=20
> One nasty problem with this is that it is not possible to figure out =
at
> compile time what the size of time_t is. You always need some sort of
> configure-time test, and an external define.
I got a notice of a pending patch for the issue:
Begin forwarded message:
From: bugzilla-daemon at llvm.org
Subject: [Bug 33638] FreeBSD head -r320347 moved TARGET_ARCH=3Dpowerpc =
to 64-bit time_t but now experimental/filesystem/operations.cpp fails =
static_asserts and such
Date: June 29, 2017 at 10:23:56 AM PDT
. . .
Comment # 2 on bug 33638 from Eric Fiselier
I have a patch for this waiting in the wings. I should be able to get to =
it
next week
=3D=3D=3D
Mark Millard
markmi at dsl-only.net
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ACD8EAE8-9D33-4E0D-B3AC-E7953544DD1F>