From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:55304)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1bLCEq-0007d9-VH
	for qemu-devel@nongnu.org; Thu, 07 Jul 2016 12:33:22 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1bLCEp-0008Rf-0z
	for qemu-devel@nongnu.org; Thu, 07 Jul 2016 12:33:19 -0400
Received: from mail-vk0-x230.google.com ([2607:f8b0:400c:c05::230]:35463)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1bLCEo-0008RV-JM
	for qemu-devel@nongnu.org; Thu, 07 Jul 2016 12:33:18 -0400
Received: by mail-vk0-x230.google.com with SMTP id v6so27227202vkb.2
	for <qemu-devel@nongnu.org>; Thu, 07 Jul 2016 09:33:18 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <5ce515ea-4a79-8b44-7271-d047461a0837@twiddle.net>
References: <1467821896-32616-1-git-send-email-rth@twiddle.net>
	<1467821896-32616-2-git-send-email-rth@twiddle.net>
	<87a8htmnj5.fsf@linaro.org>
	<5ce515ea-4a79-8b44-7271-d047461a0837@twiddle.net>
From: Peter Maydell <peter.maydell@linaro.org>
Date: Thu, 7 Jul 2016 17:32:58 +0100
Message-ID: <CAFEAcA8wz+GqwRjp8CVraZKR4RVfASBuUfi-h5tu4tthiaKiGg@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PULL v2 1/4] build: Use $(CCAS) for compiling .S
 files
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Richard Henderson <rth@twiddle.net>
Cc: =?UTF-8?B?QWxleCBCZW5uw6ll?= <alex.bennee@linaro.org>, QEMU Developers <qemu-devel@nongnu.org>

On 7 July 2016 at 17:29, Richard Henderson <rth@twiddle.net> wrote:
> On 07/07/2016 09:16 AM, Alex Benn=C3=A9e wrote:
>>>
>>> +if echo | $ccas -dM -E - | grep __clang__ > /dev/null 2>&1 ; then
>>> +  ccas=3D"$ccas -fno-integrated-as"
>>> +fi
>>
>>
>> Hi Richard,
>>
>> This looks like it breaks the Travis clang:
>>
>>     https://travis-ci.org/qemu/qemu/builds/142825178
>>
>> We get the error:
>>
>>     AS    optionrom/multiboot.o
>>     clang: error: unknown argument: '-fno-integrated-as'
>>     make[1]: *** [multiboot.o] Error 1
>>     make: *** [romsubdir-optionrom] Error 2
>>     make: *** Waiting for unfinished jobs....
>>
>> Clang in Travis is:
>>
>>     clang version 3.4 (tags/RELEASE_34/final)
>
>
> Oh ffs.  It appears -fno-integrated-as was introduced in 3.5.
>
> Is there any combination of options that allows clang 3.4 to compile the
> option roms?  I suspect that the answer is no.  And if so... what then?

...then don't build the option roms? This would just put systems
with older clang in the same boat as non-i386 systems.

thanks
-- PMM