[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 13/39] properly detect compiler in tests/Makefil
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 13/39] properly detect compiler in tests/Makefile |
Date: |
Wed, 13 Oct 2010 19:05:32 +0000 |
On Wed, Oct 13, 2010 at 7:19 AM, Paolo Bonzini <address@hidden> wrote:
> On 10/12/2010 09:04 PM, Blue Swirl wrote:
>>>
>>> diff --git a/tests/Makefile b/tests/Makefile
>>> index ff7f787..a789e2d 100644
>>> --- a/tests/Makefile
>>> +++ b/tests/Makefile
>>> @@ -64,11 +64,21 @@ linux-test: linux-test.c
>>> $(CC) $(CFLAGS) $(LDFLAGS) -o $@ $< -lm
>>>
>>> # speed test
>>> +ifeq ($(shell uname -m), x86_64)
>>> +CC_I386 = $(CC) -m32
>>> +else
>>> +ifeq ($(shell uname -m), i386)
>>> +CC_I386 = $(CC)
>>> +else
>>> +CC_I386 = i386-pc-linux-gnu-$(CC)
>>> +endif
>>> +endif
>>
>> I'd move this logic to configure.
>
> I can do that, but it seemed overengineered for a variable that's only
> needed only in one makefile rule, and furthermore a rule that will be
> invoked manually by the user. In fact I was tempted to hardcode
> "CC_I386=$(CC) -m32" which would work on x86_64 and i386 (and leave the task
> of overriding it to the user).
That part in configure could be extended to handle other targets as
well. If Alpha, ARM, CRIS, or MIPS (cross-)compilers are available, we
can easily build tests for those too. In the original Makefile
location, this would be a mess.
- Re: [Qemu-devel] [PATCH 08/39] provide portable sizeof(long) test, (continued)
[Qemu-devel] [PATCH 12/39] move --srcdir detection earlier, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 11/39] reorganize sdl-config tests, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 14/39] remove HOST_CC mention from roms/{sea, vga}bios/config.mak, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 13/39] properly detect compiler in tests/Makefile, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 15/39] let --host-cc slide into oblivion, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 16/39] introduce CFLAGS= and LDFLAGS= configure command-line options, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 18/39] add autoconfy alias CC= for --cc, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 19/39] add CPP variable, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 20/39] add autoconfy aliases MAKE=/INSTALL= for --make and --install, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 21/39] add autoconfy aliases --with-* for audio library options, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 17/39] introduce CPPFLAGS configure variable, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 22/39] make trace options use autoconfy names, Paolo Bonzini, 2010/10/12
[Qemu-devel] [PATCH 23/39] deprecate --audio-card-list, Paolo Bonzini, 2010/10/12