RFR (M): 8213996: Remove one of the SparsePRT entry tables

Thomas Schatzl thomas.schatzl at oracle.com
Wed Nov 28 10:05:22 UTC 2018


Hi,

On Wed, 2018-11-28 at 11:04 +0100, Stefan Johansson wrote:
> 
> On 2018-11-21 10:29, Thomas Schatzl wrote:
> > Hi Aleksey,
> > 
> > On Tue, 2018-11-20 at 15:32 +0100, Aleksey Shipilev wrote:
> > > On 11/20/18 3:21 PM, Thomas Schatzl wrote:
> > > >    the crashes seem unrelated as they are reproducable with
> > > > very similar patterns in older builds too (JDK-8214118).
> > > 
> > > Whoops.
> > > 
> > > > So I think I can safely reopen this review again. I fixed
> > > > Aleksey's nit and also found a few more unused methods to
> > > > remove.
> > > > 
> > > > http://cr.openjdk.java.net/~tschatzl/8213996/webrev.0_to_1/
> > > > (diff)
> > > > http://cr.openjdk.java.net/~tschatzl/8213996/webrev.1/ (full)
> 
> Looks good to me too,
> 
> Thanks for doing this cleanup,
> Stefan

  thanks for your review.

Thomas :)




More information about the hotspot-gc-dev mailing list