RFR: 8304880: [PPC64] VerifyOops code in C1 doesn't work with ZGC

Martin Doerr mdoerr at openjdk.org
Fri Mar 24 17:14:41 UTC 2023


On Fri, 24 Mar 2023 15:27:53 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> I suggest to remove this code for the following reasons:
> - It doesn't work with ZGC (oop needs to go through load barrier, see JBS issue).
> - It generates too much code. Loading oops is quite common and the oop verification code is quite lengthy.
> - Other platforms don't have it, either.

Thanks for the review!
Hmm, it may be beneficial in some bug chasing scenario if we can ensure that C1 doesn't write any broken oops. I think I had implemented the checking code for that a long time ago. On the other side, we didn't need it for quite some time. Maybe someone else has an opinion about it.

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

PR Comment: https://git.openjdk.org/jdk/pull/13175#issuecomment-1483143777


More information about the hotspot-gc-dev mailing list