[kaffe] More mipsel jit3

[Helmer Krämer]

This is a multi-part message in MIME format.

--Multipart=_Tue__9_Mar_2004_11_22_31_+0100_KnaMDPElfoOs_570
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Mon, 08 Mar 2004 13:34:13 -0800
Casey Marshall <rsdio@metastatic.org> wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> I've gone ahead and implemented translations for fcmpg, fcmpl, dcmpg,
> and dcmpl using C.cond.fmt. This gets me further in loading Kaffe (now
> System.checkPropertyAccess complains that "line.separator" is a
> zero-length key). Unfortunately, I am also getting an improper gc_free
> call, and thence a SIGSEGV, with jitBasic ControlFlowMethods.class.

You might want to remove the boolean_exception() method from
the ControlFlowMethods class (or move it to the end of the
class). It instantiates a new Throwable Object which causes
a whole bunch of methods to be translated and executed. On
arm and alpha this lead to a crash while creating a new thread
(the deleteHelper of java.io.File). This happens because the
java.lang.Thread constructor tries to access the ThreadGroup
of the current Thread which isn't created by the jit_stub.

The attached patch was an attempt to fix this, but I still
got the crash on arm. However if you're able to run the tests
with this patch, it's more likely to be a bug in the arm
jitter.

Regards,
Helmer

--Multipart=_Tue__9_Mar_2004_11_22_31_+0100_KnaMDPElfoOs_570
Content-Type: application/octet-stream;
 name="jit_stub.patch"
Content-Disposition: attachment;
 filename="jit_stub.patch"
Content-Transfer-Encoding: base64

SW5kZXg6IHRlc3QvaW50ZXJuYWwvaml0X3N0dWIuYwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvY3Zz
L2thZmZlL2thZmZlL3Rlc3QvaW50ZXJuYWwvaml0X3N0dWIuYyx2CnJldHJpZXZpbmcgcmV2aXNp
b24gMS40CmRpZmYgLXUgLXIxLjQgaml0X3N0dWIuYwotLS0gdGVzdC9pbnRlcm5hbC9qaXRfc3R1
Yi5jCTI5IFNlcCAyMDAzIDEwOjAxOjU5IC0wMDAwCTEuNAorKysgdGVzdC9pbnRlcm5hbC9qaXRf
c3R1Yi5jCTkgTWFyIDIwMDQgMTA6MTc6NDEgLTAwMDAKQEAgLTE0NCw2ICsxNDQsOCBAQAogCiAJ
ewogCQlIamF2YV9sYW5nX1RocmVhZCBtYWluVGhyZWFkOworCQlIamF2YV9sYW5nX1RocmVhZEdy
b3VwIG1haW5Hcm91cDsKKwkJSEFycmF5T2ZPYmplY3QgbWFpblRocmVhZHM7CiAJCXBhcnNlZFN0
cmluZyB0ZXN0TmFtZTsKIAkJY2hhciAqdGVzdHM7CiAJCQpAQCAtMTUzLDEwICsxNTUsMTYgQEAK
IAkJbG9hZFN0YXRpY0NsYXNzKCZqYXZhTGFuZ1Rocm93YWJsZSwgImphdmEvbGFuZy9UaHJvd2Fi
bGUiKTsKIAkJbG9hZFN0YXRpY0NsYXNzKCZqYXZhTGFuZ051bGxQb2ludGVyRXhjZXB0aW9uLCAi
amF2YS9sYW5nL051bGxQb2ludGVyRXhjZXB0aW9uIik7CiAJCWxvYWRTdGF0aWNDbGFzcygmamF2
YUxhbmdBcnJheUluZGV4T3V0T2ZCb3VuZHNFeGNlcHRpb24sICJqYXZhL2xhbmcvQXJyYXlJbmRl
eE91dE9mQm91bmRzRXhjZXB0aW9uIik7CisKKwkJbWVtc2V0KCZtYWluR3JvdXAsIDAsIHNpemVv
ZihtYWluR3JvdXApKTsKIAkJbWVtc2V0KCZtYWluVGhyZWFkLCAwLCBzaXplb2YobWFpblRocmVh
ZCkpOwotCQlqdGhyZWFkX2NyZWF0ZWZpcnN0KE1BSU5TVEFDS1NJWkUsCi0JCQkJICAgIGphdmFf
bGFuZ19UaHJlYWRfTk9STV9QUklPUklUWSwKLQkJCQkgICAgJm1haW5UaHJlYWQpOworCQlUSFJF
QURfREFUQSgpLT5qbFRocmVhZCA9ICZtYWluVGhyZWFkOworCisJCW1haW5UaHJlYWQuZ3JvdXAg
PSAmbWFpbkdyb3VwOworCisJCW1haW5Hcm91cC50aHJlYWRzID0gJm1haW5UaHJlYWRzOworCQlt
YWluVGhyZWFkcy5sZW5ndGggPSAxOworCQltYWluVGhyZWFkcy5kYXRhWzBdLmJvZHlbMF0gPSAo
SGphdmFfbGFuZ19PYmplY3QgKikmbWFpblRocmVhZDsKIAogCQlpZiggKHRlc3RzID0gZ2V0ZW52
KFRFU1RfQ0xBU1NFUykpICkKIAkJewo=

--Multipart=_Tue__9_Mar_2004_11_22_31_+0100_KnaMDPElfoOs_570--