RFR: 8255598: [PPC64] assert(Universe::heap()->is_in(result)) failed: object not in heap [v2]

Fri Nov 6 09:13:58 UTC 2020

On Thu, 5 Nov 2020 22:31:28 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> I've added a comment.

Thank you for that.

I believe `decode_raw` handles null well; `decode_raw_not_null` is there just to have an extra assertion when the caller *knows* that the oop is not null.

> Note that there's another usage with preceding null check in oopDesc::load_oop_raw.

Indeed, but I think that one is unnecessary as well, which could (or should) be addressed in this PR or another.

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

PR: https://git.openjdk.java.net/jdk/pull/1078