RFR: 8346289: Confusing phrasing in IR Framework README / User-defined Regexes
duke
duke at openjdk.org
Tue Dec 17 07:29:40 UTC 2024
On Mon, 16 Dec 2024 15:39:32 GMT, theoweidmannoracle <duke at openjdk.org> wrote:
>> If such a user-defined regex represents a not yet supported C2 IR node, it is highly encouraged to directly add a new IR node placeholder string definition to IRNode for it instead together with a static regex mapping block.
>
> The combination of "instead together" makes this sentence hard to read. I had to re-read it several times to grasp it.
@theoweidmannoracle
Your change (at version 84cc23263ea32228928ab606f2cc3b2c5f8ef518) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/22766#issuecomment-2547686159
More information about the hotspot-compiler-dev
mailing list