On Mon, 10 Aug 2026 14:35:24 GMT, Frederic Parain <[email protected]> wrote:
>> Marc Chevalier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Null-armoring > > src/hotspot/share/ci/ciInstanceKlass.cpp line 714: > >> 712: VM_ENTRY_MARK; >> 713: const Array<int>* acmp_maps = get_instanceKlass()->acmp_maps_array(); >> 714: int number_of_nonoop_entries = acmp_maps->at(0); > > `number_of_nonoop_entries` is used only in the assert below, so its > declaration and computation could be put into a `DEBUG_ONLY` or a `#ifdef > DEBUG`. Good point. Also good for the human reader, not only for gcc. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32144#discussion_r3751637969
