RFR: 8367347: Serial: Refactor CLDScanClosure [v4]
    Francesco Andreuzzi 
    fandreuzzi at openjdk.org
       
    Thu Sep 11 10:14:27 UTC 2025
    
    
  
On Thu, 11 Sep 2025 06:32:39 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
> Looks good to me. FWIW, in this and in Albert's previous patch I had to check `is_in_young_gen` to see if it was a trivial check and not something complicated. We could make it easier for the readers if this code:
> 
> ```
>         if (is_in_young_gen(new_obj) && !_has_oops_into_young_gen) {
>           _has_oops_into_young_gen = true;
>         }
> ```
> 
> were changed to:
> 
> ```
>         if (!_has_oops_into_young_gen && is_in_young_gen(new_obj)) {
>           _has_oops_into_young_gen = true;
>         }
> ```
> 
> It might also be nicer to the branch predictor.
Thanks @stefank, I addressed this in 959f932411d28c22ab101138ae9bb2027bdad990
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27199#issuecomment-3279684433
    
    
More information about the hotspot-gc-dev
mailing list