Hi, On Tue, 2018-01-30 at 07:00 +1000, David Holmes wrote: > Added in hotspot-gc-dev. Although this is in the SA it is about the > SA interaction with G1 and so likely needs someone familiar with G1 > to review it. > > David > > On 28/01/2018 10:41 PM, Yasumasa Suenaga wrote: > > PING: Could you review it? > > > > > > http://cr.openjdk.java.net/~ysuenaga/JDK-8194249/webrev.01/ > > > > This webrev has been reviewed by Jini. > > I need a Reviewer and sponsor.
looks good to me - however there is another (pre-existing) bug: the shift in that code should be a logical shift, not an arithmetic shift. I.e. ">>" instead of ">>>". I will run the patch through testing and report back in a few hours. Should be okay. Thanks, Thomas
