RFR: 8375271: [IR Framework] Rename IREncoding to ApplicableIRRules and driver/flag/test VM to Driver/Flag/Test VM [v2]
Manuel Hässig
mhaessig at openjdk.org
Wed Jan 14 12:41:22 UTC 2026
On Wed, 14 Jan 2026 12:34:33 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 two additional commits since the last revision:
>
> - Update Test VM
> - Update test/hotspot/jtreg/compiler/lib/ir_framework/TestFramework.java
>
> Co-authored-by: Manuel Hässig <manuel at haessig.org>
Thank you for addressing my comments.
-------------
Marked as reviewed by mhaessig (Committer).
PR Review: https://git.openjdk.org/jdk/pull/29229#pullrequestreview-3660540669
More information about the hotspot-compiler-dev
mailing list