Re: [Qemu-devel] spice/qlx: atomic-alignment error with clang-7

2018-09-27 Thread Peter Maydell
On 27 September 2018 at 17:59, Richard Henderson wrote: > On 9/27/18 5:53 AM, Peter Maydell wrote: >> Maybe I can do something to persuade the compiler that the >> pointer really is 4 aligned... > > Hopefully __builtin_assume_aligned(ptr, 4) is supported by that version of > clang? Yes; I sent

Re: [Qemu-devel] spice/qlx: atomic-alignment error with clang-7

2018-09-27 Thread Richard Henderson
On 9/27/18 5:53 AM, Peter Maydell wrote: > Maybe I can do something to persuade the compiler that the > pointer really is 4 aligned... Hopefully __builtin_assume_aligned(ptr, 4) is supported by that version of clang? r~

Re: [Qemu-devel] spice/qlx: atomic-alignment error with clang-7

2018-09-27 Thread Peter Maydell
On 27 September 2018 at 13:22, Peter Maydell wrote: > On 18 July 2018 at 16:12, Paolo Bonzini wrote: >> On 18/07/2018 16:56, Philippe Mathieu-Daudé wrote: >>> Using clang-7 (Debian) on x86_64 to build v3.0.0-rc1. >>> >>> Not a regression, it was probably always there: >>> >>> $ ./configure

Re: [Qemu-devel] spice/qlx: atomic-alignment error with clang-7

2018-09-27 Thread Peter Maydell
On 18 July 2018 at 16:12, Paolo Bonzini wrote: > On 18/07/2018 16:56, Philippe Mathieu-Daudé wrote: >> Using clang-7 (Debian) on x86_64 to build v3.0.0-rc1. >> >> Not a regression, it was probably always there: >> >> $ ./configure --cc=clang-7 && make hw/display/qxl.o >> hw/display/qxl.c:1884:19:

Re: [Qemu-devel] spice/qlx: atomic-alignment error with clang-7

2018-07-18 Thread Philippe Mathieu-Daudé
On 07/18/2018 12:12 PM, Paolo Bonzini wrote: > On 18/07/2018 16:56, Philippe Mathieu-Daudé wrote: >> Using clang-7 (Debian) on x86_64 to build v3.0.0-rc1. >> >> Not a regression, it was probably always there: >> >> $ ./configure --cc=clang-7 && make hw/display/qxl.o >> hw/display/qxl.c:1884:19:

Re: [Qemu-devel] spice/qlx: atomic-alignment error with clang-7

2018-07-18 Thread Paolo Bonzini
On 18/07/2018 16:56, Philippe Mathieu-Daudé wrote: > Using clang-7 (Debian) on x86_64 to build v3.0.0-rc1. > > Not a regression, it was probably always there: > > $ ./configure --cc=clang-7 && make hw/display/qxl.o > hw/display/qxl.c:1884:19: error: misaligned or large atomic operation > may

[Qemu-devel] spice/qlx: atomic-alignment error with clang-7

2018-07-18 Thread Philippe Mathieu-Daudé
Using clang-7 (Debian) on x86_64 to build v3.0.0-rc1. Not a regression, it was probably always there: $ ./configure --cc=clang-7 && make hw/display/qxl.o hw/display/qxl.c:1884:19: error: misaligned or large atomic operation may incur significant performance penalty [-Werror,-Watomic-alignment]