[jdk18] RFR: 8274323: compiler/codegen/aes/TestAESMain.java failed with "Error: invalid offset: -1434443640" after 8273297
Smita Kamath
svkamath at openjdk.java.net
Wed Dec 15 22:53:06 UTC 2021
On Tue, 14 Dec 2021 20:05:59 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
>> The failure happens with XX:+DeoptimizeAlot option. I've set reexecute bit and reset the appropriate state for the interpreter to execute the code when deoptimization occurs.
>
> Yes, we need to reexecute because code could be deoptimized during `new_array()` allocation.
> But why we allocate this temp array in Java heap? Why not on stack in stub code?
>
> Also I noticed next return from intrinsics code could be moved up before we generate new nodes in graph: `if (Matcher::htbl_entries == -1) return false;`
@vnkozlov Allocating the array in the stub will cause few changes on x86-64 side as well as change in aarch64 stubGenerator code as subkeyHtbl_48_entries will no longer be passed as an argument.
Do let me know if you think it is okay to proceed with these changes. Thank you.
-------------
PR: https://git.openjdk.java.net/jdk18/pull/19
More information about the hotspot-compiler-dev
mailing list