Re: [Qemu-devel] [PATCH 1/5] Fix i386 Host

[Jens Arm]

Hi

I tried to compile on Debian 4.0, but I get this error:

gcc -Wall -O2 -g -fno-strict-aliasing  -fno-reorder-blocks  -fno-gcse  
-fno-tree-ch  -fno-optimize-sibling-calls  -fno-crossjumping  -fno-align-labels 
 -fno-align-jumps  -fno-align-functions   -mpreferred-stack-boundary=2 
-fomit-frame-pointer   -I. -I.. -I/home/tux/compile/qemu/target-i386 
-I/home/tux/compile/qemu -MMD -MP -DNEED_CPU_H -D_GNU_SOURCE 
-D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -I/home/tux/compile/qemu/fpu  
-DHAS_AUDIO -DHAS_AUDIO_CHOICE -I/home/tux/compile/qemu/slirp -c -o op.o 
/home/tux/compile/qemu/target-i386/op.c
/home/tux/compile/qemu/target-i386/op.c: In function ‘op_fstl_ST0_A0’:
/home/tux/compile/qemu/target-i386/op.c:1925: error: unable to find a register 
to spill in class ‘GENERAL_REGS’
/home/tux/compile/qemu/target-i386/op.c:1925: error: this is the insn:
(insn:HI 55 54 56 2 ../cpu-all.h:372 (set (mem:DI (plus:SI (reg/v:SI 60 [ addr 
])
                (reg:SI 84 [ <variable>.addend ])) [0 S8 A64])
        (reg/v:DI 63 [ u ])) 80 {*movdi_2} (insn_list:REG_DEP_TRUE 54 (nil))
    (expr_list:REG_DEAD (reg/v:DI 63 [ u ])
        (expr_list:REG_DEAD (reg/v:SI 60 [ addr ])
            (expr_list:REG_DEAD (reg:SI 84 [ <variable>.addend ])
                (nil)))))
/home/tux/compile/qemu/target-i386/op.c:1925: confused by earlier errors, 
bailing out

I use "gcc (GCC) 4.1.2 20061115 (prerelease) (Debian 4.1.1-21)".

Qemu is configured like this: ./configure --disable-gcc-check 
--disable-darwin-user --disable-linux-user --target-list=i386-softmmu

Hope this helps.

Jens


On Thu, 17 Jan 2008 08:54:32 +0100
Alexander Graf <address@hidden> wrote:

> Johannes Schindelin wrote:
> > Hi,
> >
> > On Thu, 17 Jan 2008, Alexander Graf wrote:
> >
> >   
> >> This patch is mostly a cleanup of Michael Matz's patch with the ideas 
> >> that came last time included.
> >>     
> >
> > Maybe the patch subject could be "prepare i386 host for gcc4, step 1)?
> >
> >   
> 
> Basically it fixes it, just not completely ;-).
> 
> >> Index: qemu/softmmu_header.h
> >> ===================================================================
> >> --- qemu.orig/softmmu_header.h
> >> +++ qemu/softmmu_header.h
> >> @@ -189,9 +189,11 @@ static inline void glue(glue(st, SUFFIX)
> >>  #else
> >>  #error unsupported size
> >>  #endif
> >> +                  "pushl %%ecx\n"
> >>                    "pushl %6\n"
> >>                    "call %7\n"
> >>                    "popl %%eax\n"
> >> +                  "popl %%ecx\n"
> >>     
> >
> > Could you guard this so that this code is only included for gcc4?  (I am 
> > assuming that it is not needed for gcc3, as it worked so far...)
> >
> >   
> You're right on that one. Thanks for the catch.
> 
>