Request for reviews (XXL): 6893268: additional dynamic language related optimizations in C2
Thomas.Rodriguez at Sun.COM
Wed Dec 16 14:09:40 PST 2009
The only comment I'd make is that I don't think you need to the ResourceMark in is_in_package anymore and you might consider adding a VM_ENTRY_MARK at the top so that you only have to do one VM transition. Everything you are calling requires a VM transition and is implemented with GUARDED_VM_ENTRY. Otherwise I'm ok with it.
On Dec 16, 2009, at 2:30 AM, Christian Thalinger wrote:
> On Wed, 2009-12-16 at 10:04 +0100, Christian Thalinger wrote:
>> On Tue, 2009-12-15 at 14:42 -0800, John Rose wrote:
>>> FYI: Here's the code I'm playing with. But I think I prefer your startsWith method.
>> Good, then I'll roll that into the patch.
> Here is the final webrev:
> If there is no objection I will commit this tomorrow.
> -- Christian
More information about the hotspot-compiler-dev