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