Review Request: 6909153: Fix broken options on Zero

[Andrew John Hughes]

2009/12/22 Christian Thalinger <Christian.Thalinger at sun.com>:
> On Fri, 2009-12-18 at 08:38 +0000, Gary Benson wrote:
>> Hi,
>>
>> Christian Thalinger wrote:
>> > On Wed, 2009-12-09 at 16:30 +0000, Gary Benson wrote:
>> > > I've been doing a mini-audit of Zero to ensure it still works with
>> > > non-standard options.  This webrev:
>> > >
>> > >   http://cr.openjdk.java.net/~gbenson/zero-options-fixes/
>> > >
>> > > fixes the following issues:
>> > >
>> > >   1. HotSpot would segfault with -XX:+CITime if the VM is built
>> > >      with no compilers.  The patch fixes this.
>> > >
>> > >   2. The C++ interpreter doesn't support -XX:+ProfileInterpreter
>> > >      and -XX:+UseCompressedOops, and Zero doesn't support
>> > >      -XX:+TaggedStackInterpreter.  The patch disables these
>> > >      options in these configurations.
>> >
>> > Changes look good to me.
>> >
>> > > I haven't filed a bug for this, so it has no bug id yet.
>> >
>> > The CR is:
>> >
>> > 6909153: Fix broken options on Zero
>>
>> Would it be possible for someone to push this for me?
>
> Ohh, sorry, I forgot this one.  I'll do it when I find some time
> (currently having vacation).
>
> -- Christian
>
>

It's a pity external contributors can't submit JPRT commits; is there
a plan to implement this?
-- 
Andrew :-)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and the OpenJDK
http://www.gnu.org/software/classpath
http://openjdk.java.net

PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8