Request for reviews (XS): 6925573: vm/classfmt/lmt/mthnum001/mthnum00101m1/mthnum00101m1.html hangs since JDK7-b72 (-d64, solaris)

[Coleen Phillimore]

Volker,
Thank you for fixing this!  It was on my list but I hadn't gotten to it yet.

I thought that we'd want to have a special narrow_oop_compare_fn() so 
that we don't have to test if (UseCompressedOops) for each comparison, 
but your testing shows no penalty for the frequent global value test.  
To be clear though,  in your table, the -UseCompressedOops case was 
before your fix?  If so, your fix looks great, and I'd be happy to check 
it in after another review.

Thanks,
Coleen

On 05/03/10 12:57, Volker Simonis wrote:
> Hi,
>
> I fixed 6925573 which was caused by the Compressed Oops change which
> changed the method sorting algorithm for class methods during
> classloading from quicksort to bubblesort.
>
> Could somebody please review the change at
>
> http://www.progdoc.de/webrev/6925573/
>
> and submit it if is deemed to be suitable.
>
> Thanks,
> Volker
>