RFR: 8299125: UnifiedOopRef in JFR leakprofiler should treat thread local oops correctly [v2]

[Markus Grönlund]

On Mon, 9 Jan 2023 07:21:12 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

>> Maybe the shift is needed in order to handle unaligned oops, as for code blob oops? The existing root processing logic has up to now skipped code blob oops, mainly because they were hard to encode. The rationale was that they are not roots in and of themselves. Now, we have seen several instances where chains are not found on iterations, so maybe the assumption on skipping code blob oops is wrong. With this encoding scheme in place, it would be possible to pass a closure for processing the code blob oops as well.
>
> Sorry for the tardy response. 
> 
> I think I forgot what the code was actually doing at the time of writing my previous answer. We are encoding ` narrowOop*` and `oop*` here not `narrowOop`/`oop`. And `narrowOop*` are 4 byte aligned.

Thanks for the explanation, makes sense now.

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

PR: https://git.openjdk.org/jdk/pull/11741