RFR: 8358772: Template-Framework Library: Primitive Types [v3]
Emanuel Peter
epeter at openjdk.org
Thu Jun 12 11:23:54 UTC 2025
On Thu, 12 Jun 2025 10:47:56 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> review suggestions applied
>
> test/hotspot/jtreg/compiler/lib/template_framework/library/CodeGenerationDataNameType.java line 45:
>
>> 43: * @return A random constant value.
>> 44: */
>> 45: public Object con();
>
> You can remove all the public identifies since the methods are implicitly public.
good point, done :)
> test/hotspot/jtreg/compiler/lib/template_framework/library/CodeGenerationDataNameType.java line 106:
>
>> 104: * List of all {@link PrimitiveType}s.
>> 105: */
>> 106: public static final List<PrimitiveType> PRIMITIVE_TYPES = List.of(
>
> Same with the constants: They are implicitly public static final.
done!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25672#discussion_r2142406305
PR Review Comment: https://git.openjdk.org/jdk/pull/25672#discussion_r2142406768
More information about the hotspot-compiler-dev
mailing list