From owner-freebsd-current@freebsd.org  Wed Jul  8 17:04:17 2015
Return-Path: <owner-freebsd-current@freebsd.org>
Delivered-To: freebsd-current@mailman.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by mailman.ysv.freebsd.org (Postfix) with ESMTP id 9E103996395
 for <freebsd-current@mailman.ysv.freebsd.org>;
 Wed,  8 Jul 2015 17:04:17 +0000 (UTC)
 (envelope-from luigi@onelab2.iet.unipi.it)
Received: from onelab2.iet.unipi.it (onelab2.iet.unipi.it [131.114.59.238])
 by mx1.freebsd.org (Postfix) with ESMTP id 62A531D89;
 Wed,  8 Jul 2015 17:04:17 +0000 (UTC)
 (envelope-from luigi@onelab2.iet.unipi.it)
Received: by onelab2.iet.unipi.it (Postfix, from userid 275)
 id 614C17300A; Wed,  8 Jul 2015 19:05:14 +0200 (CEST)
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>
References: <CAO0mX5aTYgO1NF_Yp_eQ+dvce_1p4spG1g=pqHqhDnp5cKX+GQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAO0mX5aTYgO1NF_Yp_eQ+dvce_1p4spG1g=pqHqhDnp5cKX+GQ@mail.gmail.com>
User-Agent: Mutt/1.5.20 (2009-06-14)
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.20
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
 <freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-current>, 
 <mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current/>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
 <mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 08 Jul 2015 17:04:17 -0000

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