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

Thomas Schatzl thomas.schatzl at oracle.com
Wed Jan 29 09:29:44 UTC 2014


Hi Stefan,

On Wed, 2014-01-29 at 10:08 +0100, Stefan Karlsson wrote:

On 2014-01-29 09:50, Thomas Schatzl wrote: > Hi all, > > On Tue, 2014-01-28 at 10:29 +0100, Bengt Rutisson wrote: >> Hi Thomas, >> >> The new webrev looks much better. I like that solution. >> >> Can we do something similar with setpartialarraymask()? It would be >> good if that one could not be called with a narrowOop. Maybe this works? >> > New webrev at > http://cr.openjdk.java.net/~tschatzl/6991197/webrev.2/ > > The "fix" is to simply to only implement setpartialarraymask() for > regular oops. The compiler will automatically complain if somebody tries > to use it with narrowOops then.

Looks good.

Thanks for the review.

Thomas



More information about the hotspot-gc-dev mailing list