Re: [Qemu-devel] [PATCH] multiboot: make tests work with clang

2017-08-23 Thread Philippe Mathieu-Daudé
Hi Anatol, Next time use 'v2' for an updated patch (for the automated tracking tools), also no need to link it to the previous Message-Id it looks more messy and can get lost. On 08/23/2017 09:01 PM, Anatol Pomozov wrote: * clang 3.8 enables SSE even for 32bit code. Generate code for penti

Re: [Qemu-devel] [PATCH] multiboot: make tests work with clang

2017-08-23 Thread Anatol Pomozov
On Wed, Aug 23, 2017 at 4:44 PM, Philippe Mathieu-Daudé wrote: > Hi Anatol, > > On 08/23/2017 04:22 PM, Anatol Pomozov wrote: >> >> * explicitly disable SSE as clang enables it by default for 32bit code >> * add memset() implementation. Clang complains that the function is >> absent, gcc s

[Qemu-devel] [PATCH] multiboot: make tests work with clang

2017-08-23 Thread Anatol Pomozov
* clang 3.8 enables SSE even for 32bit code. Generate code for pentium CPU to make sure no new instructions are used. * add memset() implementation. Clang implements array zeroing in print_num() via memset() function call. --- tests/multiboot/Makefile| 2 +- tests/multiboot/libc.c

Re: [Qemu-devel] [PATCH] multiboot: make tests work with clang

2017-08-23 Thread Philippe Mathieu-Daudé
Hi Anatol, On 08/23/2017 04:22 PM, Anatol Pomozov wrote: * explicitly disable SSE as clang enables it by default for 32bit code * add memset() implementation. Clang complains that the function is absent, gcc seems provides default built-in. --- tests/multiboot/Makefile | 2 +- tests/m

[Qemu-devel] [PATCH] multiboot: make tests work with clang

2017-08-23 Thread Anatol Pomozov
* explicitly disable SSE as clang enables it by default for 32bit code * add memset() implementation. Clang complains that the function is absent, gcc seems provides default built-in. --- tests/multiboot/Makefile | 2 +- tests/multiboot/libc.c | 9 + tests/multiboot/libc.h | 2 ++