On Fri, 13 Sep 2024 12:53:29 GMT, Roman Kennke <rken...@openjdk.org> wrote:

>> src/hotspot/share/oops/oop.inline.hpp line 295:
>> 
>>> 293: // Used by scavengers
>>> 294: void oopDesc::forward_to(oop p) {
>>> 295:   assert(cast_from_oop<oopDesc*>(p) != this,
>> 
>> Do we really need the cast here?
>
> Yes, otherwise compiler complains about ambiguous != operator.

OK, we shouldn't need to. It seems like I can silence the compiler by tweaking 
oopsHierarchy.hpp. I'll deal with that as a follow-up.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20677#discussion_r1758853099

Reply via email to