Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 8 Jul 2015 19:05:14 +0200
From:      Luigi Rizzo <rizzo@iet.unipi.it>
To:        delphij@freebsd.org, freebsd-current <freebsd-current@freebsd.org>
Subject:   gcc/amd64 head build error after r281316
Message-ID:  <20150708170514.GA47276@onelab2.iet.unipi.it>
In-Reply-To: <CAO0mX5aTYgO1NF_Yp_eQ%2Bdvce_1p4spG1g=pqHqhDnp5cKX%2BGQ@mail.gmail.com>
References:  <CAO0mX5aTYgO1NF_Yp_eQ%2Bdvce_1p4spG1g=pqHqhDnp5cKX%2BGQ@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi,
the r281316 commit introduces the following lines
which break compilation with gcc on amd64 (as far as i know
immintrin.h is only available in our clang).
If there are no objections I'd like to add a further check
for the use of clang, see attached patch

Index: /home/luigi/FreeBSD/head/lib/liblzma/config.h
===================================================================
--- /home/luigi/FreeBSD/head/lib/liblzma/config.h       (revision 285281)
+++ /home/luigi/FreeBSD/head/lib/liblzma/config.h       (working copy)
@@ -150,7 +150,7 @@
 #define HAVE_ICONV 1
 
 /* Define to 1 if you have the <immintrin.h> header file. */
-#if defined(__FreeBSD__) && defined(__amd64__)
+#if defined(__clang__) && defined(__FreeBSD__) && defined(__amd64__)
 #define HAVE_IMMINTRIN_H 1
 #endif


cheers
luigi



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