RFR: 8337221: CompileFramework: test library to conveniently compile java and jasm sources for fuzzing [v13]
Emanuel Peter
epeter at openjdk.org
Tue Sep 17 17:07:50 UTC 2024
On Tue, 17 Sep 2024 07:53:19 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>>
>> move some code around
>
> test/hotspot/jtreg/compiler/lib/compile_framework/CompileFramework.java line 59:
>
>> 57: }
>> 58:
>> 59: private String sourceCodesAsString(List<SourceCode> sourceCodes) {
>
> Just a side node, since this is not C where you need to define the methods first and then use them, I suggest to do it the other way round: Define methods below as they are used. This makes it easier to read I think
Ok, I'll keep this in mind :)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20184#discussion_r1763586020
More information about the hotspot-compiler-dev
mailing list