On Thu, 12 Feb 2026 21:01:46 GMT, Stefan Karlsson <[email protected]> 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
