[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 1/4] build: Use $(CCAS) for compiling .S files
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL v2 1/4] build: Use $(CCAS) for compiling .S files |
Date: |
Thu, 7 Jul 2016 17:32:58 +0100 |
On 7 July 2016 at 17:29, Richard Henderson <address@hidden> wrote:
> On 07/07/2016 09:16 AM, Alex Bennée wrote:
>>>
>>> +if echo | $ccas -dM -E - | grep __clang__ > /dev/null 2>&1 ; then
>>> + ccas="$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
- [Qemu-devel] [PULL v2 0/4] tcg-ish updates, Richard Henderson, 2016/07/06
- [Qemu-devel] [PULL v2 2/4] tcg: Fix name for high-half register, Richard Henderson, 2016/07/06
- [Qemu-devel] [PULL v2 3/4] tcg: Optimize spills of constants, Richard Henderson, 2016/07/06
- [Qemu-devel] [PULL v2 4/4] tcg: Improve the alignment check infrastructure, Richard Henderson, 2016/07/06
- Re: [Qemu-devel] [PULL v2 0/4] tcg-ish updates, Peter Maydell, 2016/07/07