[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG |
Date: |
Sat, 13 Dec 2008 10:23:05 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Blue Swirl wrote:
> Revision: 5998
> http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=5998
> Author: blueswir1
> Date: 2008-12-13 08:16:43 +0000 (Sat, 13 Dec 2008)
>
> Log Message:
> -----------
> Fix TARGET_LONG_BITS warning in TCG
>
> Looking at tcg/tcg.c:828, the bug that the warning indicated would show up as
> incorrect PC shown in log, only on 32 bit big endian host emulating a 64 bit
> target, -d op flag enabled. Now that dyngen is gone, the patch can be applied.
>
> Modified Paths:
> --------------
> trunk/tcg/tcg-runtime.c
>
> Modified: trunk/tcg/tcg-runtime.c
> ===================================================================
> --- trunk/tcg/tcg-runtime.c 2008-12-12 20:02:52 UTC (rev 5997)
> +++ trunk/tcg/tcg-runtime.c 2008-12-13 08:16:43 UTC (rev 5998)
> @@ -29,6 +29,7 @@
>
> #include "config.h"
> #include "osdep.h"
> +#include "cpu.h" // For TARGET_LONG_BITS
> #include "tcg.h"
>
> int64_t tcg_helper_shl_i64(int64_t arg1, int64_t arg2)
>
Nice that this warning is finally fixed - but what is the point of
including all those headers at all here? [1] worked well for me while
reducing dependencies.
Jan
[1] http://permalink.gmane.org/gmane.comp.emulators.qemu/32076
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [5998] Fix TARGET_LONG_BITS warning in TCG, Blue Swirl, 2008/12/13
- [Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG,
Jan Kiszka <=
- Re: [Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG, Blue Swirl, 2008/12/13
- [Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG, Jan Kiszka, 2008/12/13
- Re: [Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG, Blue Swirl, 2008/12/13
- Re: [Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG, Blue Swirl, 2008/12/13
- [Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG, Jan Kiszka, 2008/12/13
- Re: [Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG, Blue Swirl, 2008/12/13
- [Qemu-devel] Re: [5998] Fix TARGET_LONG_BITS warning in TCG, Jan Kiszka, 2008/12/13