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