[Spice-devel] [vdagent-linux] Add -fno-strict-aliasing to default CFLAGS
Victor Toso
victortoso at redhat.com
Wed Jun 17 01:48:16 PDT 2015
Tested on rhel6, current master of vdagent build fine.
ACKed as well, pushing and closing:
https://bugs.freedesktop.org/show_bug.cgi?id=90981
On Mon, Jun 15, 2015 at 04:31:44PM +0200, Christophe Fergeau wrote:
> Compilation breaks with older gcc versions (gcc (GCC) 4.4.7 20120313
> (Red Hat 4.4.7-11) on RHEL6) with the following error (due to the
> additional use of -Werror in the default CFLAGS)
> CC src/src_spice_vdagent-vdagent-x11-randr.o
> cc1: warnings being treated as errors
> src/vdagent-x11-randr.c: In function ‘vdagent_x11_randr_handle_event’:
> src/vdagent-x11-randr.c:499: error: dereferencing pointer ‘sce’ does break strict-aliasing rules
> src/vdagent-x11-randr.c:499: error: dereferencing pointer ‘sce’ does break strict-aliasing rules
> src/vdagent-x11-randr.c:498: note: initialized from here
> make: *** [src/src_spice_vdagent-vdagent-x11-randr.o] Error 1
>
> Strict aliasing is not something we must have for the agent, and the
> alternative would most likely involve an anonymous union, or some
> casting. Given that this warning only occurs with these older compilers,
> better to go with a fix as non-invasive as possible.
>
> Resolves: https://bugs.freedesktop.org/show_bug.cgi?id=90981
> ---
> configure.ac | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index f559c2e..ca9a15b 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -130,7 +130,7 @@ fi
>
> # If no CFLAGS are set, set some sane default CFLAGS
> if test "$ac_test_CFLAGS" != set; then
> - DEFAULT_CFLAGS="-Wall -Werror -Wp,-D_FORTIFY_SOURCE=2 -fstack-protector --param=ssp-buffer-size=4"
> + DEFAULT_CFLAGS="-Wall -Werror -Wp,-D_FORTIFY_SOURCE=2 -fno-strict-aliasing -fstack-protector --param=ssp-buffer-size=4"
> for F in $DEFAULT_CFLAGS; do
> AC_MSG_CHECKING([whether $CC supports $F])
> save_CFLAGS="$CFLAGS"
> --
> 2.4.3
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
More information about the Spice-devel
mailing list