Request for test change

2011-04-26 Thread Gary Benson
Hi all, The tests in jdk/test/java/dyn have a checkImplementedPlatform() method which disables the tests on platforms other than x86 and Sparc. This webrev adds the zero-assembler port to the list of supported platforms: http://cr.openjdk.java.net/~gbenson/zero-jsr292-tests/ Would it be possi

Review Request: Zero JSR 292 support

2011-04-26 Thread Gary Benson
Hi all, This webrev adds support for JSR 292 to Zero: http://cr.openjdk.java.net/~gbenson/zero-jsr292-01/ Note that is is designed to apply after the fix for 7032458, which is currently under review and can be found here: http://cr.openjdk.java.net/~gbenson/zero-shark-fixes-04-2/ I don't h

Re: Review Request: Zero JSR 292 support

2011-04-11 Thread Gary Benson
Christian Thalinger wrote: > On Apr 5, 2011, at 3:46 PM, Gary Benson wrote: > > Christian Thalinger wrote: > > > On Apr 4, 2011, at 10:34 AM, Gary Benson wrote: > > > > John Rose wrote: > > > > > On Apr 1, 2011, at 7:33 AM, Gary Benson wrote: > &

Re: Review Request: Zero JSR 292 support

2011-04-05 Thread Gary Benson
Christian Thalinger wrote: > On Apr 5, 2011, at 4:32 PM, Gary Benson wrote: > > Christian Thalinger wrote: > > > On Apr 5, 2011, at 4:04 PM, Gary Benson wrote: > > > > Christian Thalinger wrote: > > > > > hotspot/src

Re: Review Request: Zero JSR 292 support

2011-04-05 Thread Gary Benson
Christian Thalinger wrote: > On Apr 5, 2011, at 4:04 PM, Gary Benson wrote: > > Christian Thalinger wrote: > > > hotspot/src/share/vm/interpreter/bytecodeInterpreter.cpp: > > > > > > + assert(false, "Should have thrown incompatible class change

Re: Review Request: Zero JSR 292 support

2011-04-05 Thread Gary Benson
Christian Thalinger wrote: > On Apr 1, 2011, at 4:33 PM, Gary Benson wrote: > > This webrev adds support for JSR 292 to Zero: > > > > http://cr.openjdk.java.net/~gbenson/zero-jsr292-01/ > > hotspot/src/cpu/zero/vm/cppInterpreter_zero.cpp: > > +

Re: Review Request: Zero JSR 292 support

2011-04-05 Thread Gary Benson
Christian Thalinger wrote: > On Apr 4, 2011, at 10:34 AM, Gary Benson wrote: > > John Rose wrote: > > > On Apr 1, 2011, at 7:33 AM, Gary Benson wrote: > > > > This webrev adds support for JSR 292 to Zero: > > > > > > > > http://cr.openj

Re: Review Request: Zero JSR 292 support

2011-04-04 Thread Gary Benson
John Rose wrote: > On Apr 1, 2011, at 7:33 AM, Gary Benson wrote: > > This webrev adds support for JSR 292 to Zero: > > > > http://cr.openjdk.java.net/~gbenson/zero-jsr292-01/ > > Most impressive! I think this matches the following previously > filed bug: > &g