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

Paul Hohensee phh at openjdk.org
Thu Dec 11 18:22:46 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!

Marked as reviewed by phh (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/28748#pullrequestreview-3568736004


More information about the hotspot-compiler-dev mailing list