RFR (S): JDK-6991197 G1: specialize deal_with_reference() for narrowOop*

Thomas Schatzl thomas.schatzl at oracle.com
Mon Jan 27 19:27:42 UTC 2014


Hi,

On Mon, 2014-01-27 at 14:15 +0100, Stefan Karlsson wrote:
> Thomas,
> 
> On 2014-01-27 12:51, Thomas Schatzl wrote:
> > Hi all,
> >
> > Testing:
> > jprt, specjbb2005/13, specjvm98/2008, CRM Fuse
> >
> > Bug entry:
> > https://bugs.openjdk.java.net/browse/JDK-6991197
> >
> > Webrev:
> > http://cr.openjdk.java.net/~tschatzl/6991197/webrev/
> 
> Have you considered specializing has_partial_array_mask() to always 
> return false when T = narrowOop? If you do it that way, you would get 
> rid of the code duplication.

  as mentioned, that looks a lot better to me. Thanks for the
suggestion.

New webrev at http://cr.openjdk.java.net/~tschatzl/6991197/webrev.1/

Testing:
jprt

Thomas




More information about the hotspot-gc-dev mailing list