[lworld] RFR: 8377451: [lworld] Add ValuePayload abstraction [v8]
Stefan Karlsson
stefank at openjdk.org
Thu Feb 12 21:07:03 UTC 2026
On Thu, 12 Feb 2026 21:01:46 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
>> src/hotspot/share/interpreter/zero/bytecodeInterpreter.cpp line 522:
>>
>>> 520: #ifdef USELABELS
>>> 521: const static void* const opclabels_data[256] = {
>>> 522: /* 0x00 */
>>
>> Was this change necessary?
>
> That's what Axel refers to here:
> https://github.com/openjdk/valhalla/pull/2068#issuecomment-3883935914
Debug builds uses a normal C++ switch construct but release builds use an ad-hoc dispatch table instead. That dispatch table doesn't work without this fix, and this caused release versions of zero to not even build.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/2068#discussion_r2801055827
More information about the valhalla-dev
mailing list