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:33:54 UTC 2024


On Tue, 17 Sep 2024 09:14: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:
>> 
>>   move some code around
>
> Thanks for the updates - already looks much better! Some more comments.

@chhagedorn thanks a lot for the second round of review! I finally learned how to to Javadocs, and how to compile without IDE:

`jdk-fork2/open/test/hotspot/jtreg$ javadoc -sourcepath . -d ./docs -subpackages compiler.lib.compile_framework`

> test/hotspot/jtreg/testlibrary_tests/compile_framework/examples/TestFrameworkJavaExample.java line 49:
> 
>> 47:  * classes (see {@code getEscapedClassPathOfCompiledClasses}).
>> 48:  */
>> 49: public class TestFrameworkJavaExample {
> 
> Suggestion: This might be better named `IRFrameworkJavaExample`. It is a little bit unfortunate that I named the IR Framework main class `TestFramework` and not `IRFramework`...but I guess it's too late for that now.

Yeah, unfortunate 😅

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

PR Comment: https://git.openjdk.org/jdk/pull/20184#issuecomment-2356513651
PR Review Comment: https://git.openjdk.org/jdk/pull/20184#discussion_r1763620758


More information about the hotspot-compiler-dev mailing list