Re: [Qemu-devel] [PATCH] risu-m68k: update fpregs

2017-02-20 Thread Peter Maydell
On 20 February 2017 at 13:08, Laurent Vivier wrote: > Le 20/02/2017 à 13:49, Peter Maydell a écrit : >> Oops, that was unintended. We definitely don't want to require >> decorating all the function calls in the .risu files. Reverting >> commit 6a3647ae8918 should fix this, but I'll see if there's

Re: [Qemu-devel] [PATCH] risu-m68k: update fpregs

2017-02-20 Thread Laurent Vivier
Le 20/02/2017 à 13:49, Peter Maydell a écrit : > On 20 February 2017 at 12:41, Laurent Vivier wrote: >> I have some problems with risugen since some functions have been moved >> to common: >> >> $ ./risugen --numinsns 1 --pattern ABCD m68k.risu ABCD.out >> Generating code using patterns: ABCD

Re: [Qemu-devel] [PATCH] risu-m68k: update fpregs

2017-02-20 Thread Peter Maydell
On 20 February 2017 at 12:41, Laurent Vivier wrote: > I have some problems with risugen since some functions have been moved > to common: > > $ ./risugen --numinsns 1 --pattern ABCD m68k.risu ABCD.out > Generating code using patterns: ABCD M68000... > Syntax error detected evaluating ABCD M680

Re: [Qemu-devel] [PATCH] risu-m68k: update fpregs

2017-02-20 Thread Laurent Vivier
Le 20/02/2017 à 13:14, Peter Maydell a écrit : > On 19 February 2017 at 20:02, Laurent Vivier wrote: >> f_fpregs is a 2d array, not 1d: >> >> typedef struct fpregset >> { >>int f_pcr; >>int f_psr; >>int f_fpiaddr; >> #ifdef __mcoldfire__ >>int f_fpregs[8][2]; >> #else >>int

Re: [Qemu-devel] [PATCH] risu-m68k: update fpregs

2017-02-20 Thread Peter Maydell
On 19 February 2017 at 20:02, Laurent Vivier wrote: > f_fpregs is a 2d array, not 1d: > > typedef struct fpregset > { >int f_pcr; >int f_psr; >int f_fpiaddr; > #ifdef __mcoldfire__ >int f_fpregs[8][2]; > #else >int f_fpregs[8][3]; > #endif > } fpregset_t; > > For the mome

[Qemu-devel] [PATCH] risu-m68k: update fpregs

2017-02-19 Thread Laurent Vivier
f_fpregs is a 2d array, not 1d: typedef struct fpregset { int f_pcr; int f_psr; int f_fpiaddr; #ifdef __mcoldfire__ int f_fpregs[8][2]; #else int f_fpregs[8][3]; #endif } fpregset_t; For the moment, we don't manage ColdFire case, only 680x0. Signed-off-by: Laurent Vivier --