RFR: 8276098: Do precise BOT updates in G1 evacuation phase [v3]

Stefan Johansson sjohanss at openjdk.java.net
Tue Nov 9 09:03:37 UTC 2021


On Mon, 8 Nov 2021 15:30:41 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Stefan Johansson has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Thomas review
>
> src/hotspot/share/gc/g1/g1Allocator.hpp line 159:
> 
>> 157:   G1BotUpdatingPLAB(size_t word_sz) : PLAB(word_sz) { }
>> 158:   // Sets the new PLAB buffer as well as updates the threshold and region.
>> 159:   virtual void set_buf(HeapWord* buf, size_t word_sz);
> 
> Is `override` better here?

I can change this to `override` if you prefer. I see that we have some uses of it in G1 already.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6166



More information about the hotspot-gc-dev mailing list