RFR: 8373428: Refine variables with the same name in nested scopes in PhaseChaitin::gather_lrg_masks
Hamlin Li
mli at openjdk.org
Fri Dec 12 10:03:20 UTC 2025
On Thu, 11 Dec 2025 18:19:40 GMT, Paul Hohensee <phh 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).
@phohensee Thank you!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28748#issuecomment-3645777468
More information about the hotspot-compiler-dev
mailing list