RFR: 8375271: [IR Framework] Rename IREncoding to ApplicableIRRules and driver/flag/test VM to Driver/Flag/Test VM [v3]
Tobias Hartmann
thartmann at openjdk.org
Mon Jan 19 13:51:36 UTC 2026
On Mon, 19 Jan 2026 08:25:49 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> This patch is part of a series of patches split of from a prototype implementation for replacing the hotspot-pid file based transfer of IR dumps between the Test VM and the Driver VM with a socket based transfer (see [JDK-8375271](https://bugs.openjdk.org/browse/JDK-8375271) for more information). This should ease reviews.
>>
>> This patch is about naming updates:
>>
>> `IREncoding` is quite generic and does not really tell what it is about by looking at its name. I suggest to rename it to `ApplicableIRRules` to better reflect what it is about. I also capitalized the first letter of "driver/flag/test VM" to better indicate that these are proper names.
>>
>> Thanks,
>> Christian
>
> Christian Hagedorn has updated the pull request incrementally with one additional commit since the last revision:
>
> Update test/hotspot/jtreg/compiler/lib/ir_framework/flag/FlagVM.java
>
> Co-authored-by: Damon Fenacci <damon.fenacci at oracle.com>
Looks good to me too.
-------------
Marked as reviewed by thartmann (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/29229#pullrequestreview-3678181430
More information about the hotspot-compiler-dev
mailing list