On Fri, 14 Aug 2026 14:16:24 GMT, Frederic Parain <[email protected]> wrote:
>> Marc Chevalier has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Remove useless region
>
> src/hotspot/share/opto/library_call.cpp line 5648:
>
>> 5646: Node* klass_header_addr =
>> off_heap_plus_addr(load_mirror_from_klass(obj_klass),
>> oopDesc::mark_offset_in_bytes());
>> 5647: Node* klass_header = make_load(no_ctrl, klass_header_addr,
>> TypeX_X, TypeX_X->basic_type(), MemNode::unordered);
>> 5648: if (!stopped()) {
>
> I'm not sure the control can be killed between here and the previous
> if(!stopped()) check at line 5642.
That's correct. It's a leftover from the merge with [JDK-8389325: Remove the
UseObjectMonitorTable flag and related
code](https://bugs.openjdk.org/browse/JDK-8389325) that removes a
`hashcode_is_safe_to_read` that was in between, but now it can be safely
removed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32144#discussion_r3784687363
