Integrated: 8373428: Refine variables with the same name in nested scopes in PhaseChaitin::gather_lrg_masks

Hamlin Li mli at openjdk.org
Mon Dec 15 09:13:47 UTC 2025


On Wed, 10 Dec 2025 14:43:04 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you help to review this trivial patch?
> 
> In PhaseChaitin::gather_lrg_masks, several variables have the same name in nested scopes, it looks like following code snippet.
> {
>   A a;
>   {
>     A a;
>   }
> }
> 
> This is not helpful to code readability, in particular in a long method like `gather_lrg_masks`, better to rename them.
> 
> Thanks!

This pull request has now been integrated.

Changeset: 3559eeca
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/3559eeca0edd537c6160c6753cf6fc304afee4ca
Stats:     20 lines in 1 file changed: 0 ins; 0 del; 20 mod

8373428: Refine variables with the same name in nested scopes in PhaseChaitin::gather_lrg_masks

Reviewed-by: phh

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

PR: https://git.openjdk.org/jdk/pull/28748


More information about the hotspot-compiler-dev mailing list