On Thu, 26 Jan 2023 19:30:50 GMT, Christoph <[email protected]> wrote:
> We readded the splitting code and the test is passing. In the test we could
> make it work with up to 130 modules where each module _n_ requires all
> modules from _0...n_
The patch looks similar to the original. As a quick test, I changed the
threshold to 10 (from 90) to force the creation of helper methods and it fails
as expected with:
$ java -Xlog:init=debug -XX:+UnlockDiagnosticVMOptions
-XX:+BytecodeVerificationLocal -version
Error occurred during initialization of boot layer
java.lang.VerifyError: Bad local variable type
Exception Details:
Location:
jdk/internal/module/SystemModules$default.moduleDescriptorsSub1([Ljava/lang/module/ModuleDescriptor;)V
@18: aload
Reason:
Type top (current frame, locals[14]) is not assignable to reference type
Current Frame:
bci: @18
flags: { }
locals: { 'jdk/internal/module/Builder',
'[Ljava/lang/module/ModuleDescriptor;' }
stack: { 'jdk/internal/module/Builder', 'jdk/internal/module/Builder',
'[Ljava/lang/module/ModuleDescriptor$Requires;',
'[Ljava/lang/module/ModuleDescriptor$Requires;', integer }
Do you want to continue with this issue?
-------------
PR: https://git.openjdk.org/jdk/pull/10704
