RFR: 8280473: CI: Support unresolved JVM_CONSTANT_Dynamic constant pool entries
Dean Long
dlong at openjdk.java.net
Thu Feb 10 22:54:10 UTC 2022
On Thu, 10 Feb 2022 11:37:18 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
> Properly support unresolved CONSTANT_Dynamic CP entries in C1/C2.
>
> Right now, both compilers bail out the compilation when they encounter a single unresolved dynamic constant. It forces users to do eager resolution which defeats the whole purpose of using dynamic constants.
>
> The patch is focused on C1 and enables code patching for dynamic constants, but there are minor C2-specific changes to avoid compilation bailouts there.
>
> Testing: hs-tier1 - hs-tier4
src/hotspot/share/c1/c1_InstructionPrinter.cpp line 265:
> 263:
> 264: void InstructionPrinter::print_unsafe_op(UnsafeOp* op, const char* name) {
> 265: output()->print("%s(", name);
Do we need a matching ')' somewhere?
-------------
PR: https://git.openjdk.java.net/jdk/pull/7423
More information about the hotspot-compiler-dev
mailing list