RFR: 8367531: Template Framework: use scopes and tokens instead of misbehaving immediate-return-queries [v29]
Roberto Castañeda Lozano
rcastanedalo at openjdk.org
Mon Nov 17 15:18:54 UTC 2025
On Thu, 13 Nov 2025 07:43:53 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> inflate abreviations to full names
>
> test/hotspot/jtreg/compiler/lib/template_framework/TemplateFrame.java line 53:
>
>> 51: * has such a set of hashtag replacements, and implicitly provides access to the
>> 52: * hashtag replacements of the outer {@link TemplateFrame}s, up to the outermost
>> 53: * of the current {@link Template}. If a hashtag replacemnt is added in a scope,
>
> Suggestion:
>
> * of the current {@link Template}. If a hashtag replacement is added in a scope,
@eme64 please apply this suggestion.
> test/hotspot/jtreg/compiler/lib/template_framework/TemplateFrame.java line 54:
>
>> 52: * hashtag replacements of the outer {@link TemplateFrame}s, up to the outermost
>> 53: * of the current {@link Template}. If a hashtag replacemnt is added in a scope,
>> 54: * we have to find traverse to outer scopes until we find one that is not transparent
>
> Suggestion:
>
> * we have to traverse to outer scopes until we find one that is not transparent
@eme64 please apply this suggestion.
> test/hotspot/jtreg/compiler/lib/template_framework/TemplateFrame.java line 65:
>
>> 63: * on how deeply nested the code is at a given point, correlating to the runtime that
>> 64: * would be spent if the code was executed. The idea is that once the fuel is depleated,
>> 65: * we do not want to nest more deaply, so that there is a reasonable chance that the
>
> Suggestion:
>
> * we do not want to nest more deeply, so that there is a reasonable chance that the
@eme64 please apply this suggestion.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27255#discussion_r2534468193
PR Review Comment: https://git.openjdk.org/jdk/pull/27255#discussion_r2534468724
PR Review Comment: https://git.openjdk.org/jdk/pull/27255#discussion_r2534469828
More information about the hotspot-compiler-dev
mailing list