cpu-exec: Fix compiler warning (-Werror=clobbered)
commit0448f5f8b816923b198ab6c32286fd1f3b2f3e45
authorStefan Weil <sw@weilnetz.de>
Sat, 26 Sep 2015 11:23:26 +0000 (26 13:23 +0200)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 4 Nov 2015 14:56:04 +0000 (4 15:56 +0100)
tree2d8f71f198893a41c26018e0cdc509ab023d76b4
parent680a4783dc13f1059c03d11da58193d76c19ead6
cpu-exec: Fix compiler warning (-Werror=clobbered)

Reloading of local variables after sigsetjmp is only needed for some
buggy compilers.

The code which should reload these variables causes compiler warnings
with gcc 4.7 when compiler optimizations are enabled:

cpu-exec.c:204:15: error:
 variable ‘cpu’ might be clobbered by ‘longjmp’ or ‘vfork’ [-Werror=clobbered]
cpu-exec.c:207:15: error:
 variable ‘cc’ might be clobbered by ‘longjmp’ or ‘vfork’ [-Werror=clobbered]
cpu-exec.c:202:28: error:
 argument ‘env’ might be clobbered by ‘longjmp’ or ‘vfork’ [-Werror=clobbered]

Now this code is only used for compilers which need it
(and gcc 4.5.x, x > 0 which does not need it but won't give warnings).

There were bug reports for clang and gcc 4.5.0, while gcc 4.5.1
was reported to work fine without the reload code. For clang it
is not clear which versions are affected, so simply keep the status quo
for all clang compilations. This can be improved later.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Message-Id: <1443266606-21400-1-git-send-email-sw@weilnetz.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
cpu-exec.c