Date: Sun, 22 Mar 2015 22:29:43 +0100 From: Dimitry Andric <dim@FreeBSD.org> To: Craig Rodrigues <rodrigc@FreeBSD.org> Cc: "freebsd-testing@freebsd.org" <freebsd-testing@freebsd.org>, freebsd-current Current <freebsd-current@freebsd.org>, "jenkins-admin@freebsd.org" <jenkins-admin@freebsd.org>, freebsd-toolchain@freebsd.org Subject: Re: Jenkins build is still unstable: FreeBSD_HEAD-tests2 #867 Message-ID: <494AEF4B-0AF8-449A-9B41-9AC4F4552AF0@FreeBSD.org> In-Reply-To: <BE5A3694-E1D1-4A47-8707-D6214E997363@FreeBSD.org> References: <1669399171.13.1427029129760.JavaMail.jenkins@jenkins-9.freebsd.org> <799490341.14.1427048792932.JavaMail.jenkins@jenkins-9.freebsd.org> <CAG=rPVeyamLPnC5i05_=Ub0D%2BV256U_8t8R5tfDitC=NkjBOFQ@mail.gmail.com> <BE5A3694-E1D1-4A47-8707-D6214E997363@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] On 22 Mar 2015, at 22:23, Dimitry Andric <dim@FreeBSD.org> wrote: > > On 22 Mar 2015, at 22:02, Craig Rodrigues <rodrigc@FreeBSD.org> wrote: >> >> On Sun, Mar 22, 2015 at 11:26 AM, <jenkins-admin@freebsd.org> wrote: >> >>> See <https://jenkins.freebsd.org/job/FreeBSD_HEAD-tests2/867/>; >>> >> >> Can someone with toolchain expertise look at this? >> After the clang 3.6.1 import, /bin/expr behaves differently. >> >> With clang 3.5.0: >> >> # expr 4611686018427387904 + 4611686018427387904 >> expr: overflow >> >> With clang 3.6.1: >> >> # expr 4611686018427387904 + 4611686018427387904 >> -9223372036854775808 > > It works fine for me: > > $ /usr/obj/usr/src/bin/expr/expr 4611686018427387904 + 4611686018427387904 > expr: overflow Ah right, that was on i386, on amd64 it does result in -2^63. It is indeed caused by reliance on signed integer wrapping. This diff should fix it, without rewriting the utility: Index: bin/expr/Makefile =================================================================== --- bin/expr/Makefile (revision 280156) +++ bin/expr/Makefile (working copy) @@ -6,6 +6,9 @@ PROG= expr SRCS= expr.y YFLAGS= +# expr relies on signed integer wrapping +CFLAGS+= -fwrapv + NO_WMISSING_VARIABLE_DECLARATIONS= .if ${MK_TESTS} != "no" -Dimitry [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.27 iEYEARECAAYFAlUPNEcACgkQsF6jCi4glqP5HwCeKYgKWHDn47Xr4qJQROpKmJo5 jxYAoO+QWs7JjiiTgzGMaXmRrepUcuHD =pRHL -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?494AEF4B-0AF8-449A-9B41-9AC4F4552AF0>