RFR: 8264268: Don't use oop types for derived pointers

[John R Rose]

On Fri, 26 Mar 2021 11:52:58 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> The JIT compiler embeds pointers to addresses within an object. These are called derived pointers. When the GC moves objects, these pointers need to be updated explicitly, because the GC only deals with the real oops of the objects (base pointer).
> 
> The code that deals with this uses oop* for the address containing the base pointer. This is fine, the address contains an oop. However, it also uses oop* for the interior pointer, even though the contents is not a valid oop.
> 
> This creates temporary oops that does not conform to the normal requirements for oops. For example, the lower three bits could be set. This makes it problematic to write stricter verification code.
> 
> I propose that we use intptr_t* instead of oop*, and only use oop* when the location is known to contain a valid oop.

Looks good.

Maybe add `typedef intptr_t derived_oop_t` to make the operations on d-oops a little more distinctive.

Maybe add a static assert that `sizeof(oop) == sizeof(intptr_t)`, just in case there's a problem with compressed oops in the future.

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

Marked as reviewed by jrose (Reviewer).

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