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