RFR: 8337221: CompileFramework: test library to conveniently compile java and jasm sources for fuzzing [v17]

Christian Hagedorn chagedorn at openjdk.org
Mon Sep 23 13:30:15 UTC 2024


On Mon, 23 Sep 2024 12:32:16 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/lib/compile_framework/CompileFramework.java line 48:
>> 
>>> 46:      * Set up a new Compile Framework instance, for a new compilation unit.
>>> 47:      */
>>> 48:     public CompileFramework() {}
>> 
>> You can probably omit that since it's empty.
>
> I wanted to hava a javadoc string for the constructor though. Let me know if you think I should remove it.

I see, I guess then it's fine.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20184#discussion_r1771419780


More information about the hotspot-compiler-dev mailing list