6433829: C interpreter bug fixes from licensee

[Coleen Phillimore]

Gary, thanks for looking into it.  There was a 64 bit assembler port of 
this C++ interpreter once for another platform, but the code might be 
unused now.
Thanks,
Coleen

Gary Benson wrote:
> 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
>>     
>
>