[kaffe] Re: jit3 expert wanted! (again)

[Helmer Krämer]

On Fri, 23 Jul 2004 17:49:06 +0900 (JST)
Kiyo Inaba <inaba at src.ricoh.co.jp> wrote:

Hi again,

> And again, the two move instructions at 0x293478 and 0x29347c destroy
> the attempt made at 0x293476...
> 
> I think similar modification what Helmer showed may solve this problem,
> but I can not figure out where exactly I have to modify.

after some more thinking about this, I think we can remove
the entire if(), as is done in the attached patch:

Before generating code for a bytecode instruction, we check
whether or not an exception raised during the execution of
that instruction can be caught. Therefore, we can determine
the necessary spills (e.g. when invoking a method or before
a throw statement) in icode.c and generate them there. Since
icode.c already contains the necessary code for this, we can
savely remove the if(), since the spills generated in the if
are already generated by icode.c.

With this patch applied, kaffe still passes the general
regression tests as well as the jit specific tests (with
some additional tests from myself), so it can't be that
bad.

No I'm only hoping that this fixes the m68k backend ....

Regards,
Helmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jit3.patch
Type: application/octet-stream
Size: 783 bytes
Desc: not available
Url : http://kaffe.org/pipermail/kaffe/attachments/20040724/8354d819/attachment-0002.obj