6433829: C interpreter bug fixes from licensee

[Gary Benson]

Hi,

I tried to add a comment to the bug, but it won't let me log in so
I'll post it here instead:

The old LOCALS_INT code is still there in x86, but not in zero or
sparc.  However, I don't think that there is assembler support for
the C++ interpreter for 64-bit x86, so it would only become an
issue were this ever written.

I can't see the UseHeavyMonitors stuff in bytecodeInterpreter.cpp,
so I'm guessing this is still an issue, but I would expect that code
would need adding in more cases than just this( specifically, the
native entry point would also need extending) so this would seem to
be an incomplete fix at first glance.

Cheers,
Gary

Coleen Phillimore wrote:
> Hi, There are a couple of fixes to the C++ interpreter sent in a
> long time ago. The files have moved a little but could someone from
> the openjdk hotspot community who uses the Hotspot c++ interpreter
> evaluate this and submit a patch if this bug affects them?
>
> http://bugs.sun.com/view_bug.do?bug_id=6433829
>
> Thanks,
> Coleen

-- 
http://gbenson.net/