Re: [PATCH] vm: introduce per vm_class access flags.

2009-08-17 Thread Tomek Grabiec
2009/8/17 Vegard Nossum : > 2009/8/17 Pekka Enberg : >> On Mon, 2009-08-17 at 19:36 +0200, Tomek Grabiec wrote: >>> 2009/8/17 Pekka Enberg : >>> > On Mon, 2009-08-17 at 15:38 +0200, Tomek Grabiec wrote: >>> >> We need to assign access flags for arrays and primitive classes which >>> >> have no corr

Re: [PATCH] vm: introduce per vm_class access flags.

2009-08-17 Thread Vegard Nossum
2009/8/17 Pekka Enberg : > On Mon, 2009-08-17 at 19:36 +0200, Tomek Grabiec wrote: >> 2009/8/17 Pekka Enberg : >> > On Mon, 2009-08-17 at 15:38 +0200, Tomek Grabiec wrote: >> >> We need to assign access flags for arrays and primitive classes which >> >> have no corresponding cafebabe_class. Therefo

[PATCH] linear-scan: Fix range end checks

2009-08-17 Thread Pekka Enberg
From: Pekka Enberg The ->end member of struct live_range is not part of the range. Fix up some range end checks in jit/linear-scan.c which assume ->end is part of the range. Cc: Arthur HUILLET Signed-off-by: Pekka Enberg --- Arthur, I'd appreciate if you had a look at this patch before I go ah

[penberg/jato] 23c359: vm: introduce per vm_class access flags.

2009-08-17 Thread noreply
Branch: refs/heads/master Home: http://github.com/penberg/jato Commit: 23c359769f002feca749f0bd083962982a5751f6 http://github.com/penberg/jato/commit/23c359769f002feca749f0bd083962982a5751f6 Author: Tomek Grabiec Date: 2009-08-17 (Mon, 17 Aug 2009) Changed paths: M Makefile M includ

Re: [PATCH] vm: introduce per vm_class access flags.

2009-08-17 Thread Pekka Enberg
On Mon, 2009-08-17 at 19:36 +0200, Tomek Grabiec wrote: > 2009/8/17 Pekka Enberg : > > On Mon, 2009-08-17 at 15:38 +0200, Tomek Grabiec wrote: > >> We need to assign access flags for arrays and primitive classes which > >> have no corresponding cafebabe_class. Therefore ->access_flags field is > >>

Re: [PATCH] vm: introduce per vm_class access flags.

2009-08-17 Thread Tomek Grabiec
2009/8/17 Pekka Enberg : > On Mon, 2009-08-17 at 15:38 +0200, Tomek Grabiec wrote: >> We need to assign access flags for arrays and primitive classes which >> have no corresponding cafebabe_class. Therefore ->access_flags field is >> introduced for struct vm_class. >> >> Signed-off-by: Tomek Grabie

Re: [PATCH] vm: introduce per vm_class access flags.

2009-08-17 Thread Pekka Enberg
On Mon, 2009-08-17 at 15:38 +0200, Tomek Grabiec wrote: > We need to assign access flags for arrays and primitive classes which > have no corresponding cafebabe_class. Therefore ->access_flags field is > introduced for struct vm_class. > > Signed-off-by: Tomek Grabiec I'd rather we didn't make s

[PATCH] vm: introduce per vm_class access flags.

2009-08-17 Thread Tomek Grabiec
We need to assign access flags for arrays and primitive classes which have no corresponding cafebabe_class. Therefore ->access_flags field is introduced for struct vm_class. Signed-off-by: Tomek Grabiec --- Makefile |1 + include/vm/class.h| 17 + r

[penberg/jato] ff14b7: jit: fix expression type for EXPR_INVOKE*

2009-08-17 Thread noreply
Branch: refs/heads/master Home: http://github.com/penberg/jato Commit: ff14b74784ad0103efa0a33d812d6c06c7b3c615 http://github.com/penberg/jato/commit/ff14b74784ad0103efa0a33d812d6c06c7b3c615 Author: Pekka Enberg Date: 2009-08-17 (Mon, 17 Aug 2009) Changed paths: M Makefile M include

[PATCH 1/2] jit: introduce mimic_stack_type() helper

2009-08-17 Thread Tomek Grabiec
It returns the type for expression which should be pushed on mimic stack. Types: byte, boolean, char, short should be converted to int. Signed-off-by: Tomek Grabiec --- include/jit/expression.h |9 + jit/expression.c |9 ++--- 2 files changed, 11 insertions(+), 7 dele

[PATCH 2/2] regression: introduce jvm/InvokeResultTest.j

2009-08-17 Thread Tomek Grabiec
Signed-off-by: Tomek Grabiec --- Makefile |3 +- regression/jvm/InvokeResultTest.j | 89 + regression/run-suite.sh |1 + 3 files changed, 92 insertions(+), 1 deletions(-) create mode 100644 regression/jvm/InvokeRes

[PATCH] test: fix 'make test' breakage

2009-08-17 Thread Tomek Grabiec
Signed-off-by: Tomek Grabiec --- test/jit/invoke-bc-test.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/test/jit/invoke-bc-test.c b/test/jit/invoke-bc-test.c index 01a7ff7..6d8c7d4 100644 --- a/test/jit/invoke-bc-test.c +++ b/test/jit/invoke-bc-test.c @@ -320,7 +

[PATCH] jit: fix expression type for EXPR_INVOKE*

2009-08-17 Thread Tomek Grabiec
When method returns char, byte, boolean or short then int value should be pushed on mimic stack. Primitive types listed above do not occur in java stack. They are always converted to int implicitly. This caused failed assertion for the following method: jato: jit/typeconv-bc.c:56: convert_truncat

[JATO] method resolving failure

2009-08-17 Thread Tomek Grabiec
Hi, I'd like to report a bug, for which I'm not sure how proper solution should look like. Below is a test case for which I get the following error, as far as I can tell it's because method x() can not be found in abstract class A: convert_invokevirtual: warning: unable to resolve invocation targ