RFR: 8315890: Attempts to load from nullptr in instanceKlass.cpp and unsafe.cpp [v2]

[Matias Saavedra Silva]

On Mon, 30 Oct 2023 22:46:48 GMT, David Holmes <dholmes at openjdk.org> wrote:

> Not sure why it needed to be lifted out of Unsafe. The issue description should be updated now.

Sorry, I meant to explain this change in response to your comments. I'll explain here:

The call stack that results in the issue shown in `get_volatile()` starts near where the new assert is placed. When discussing with @coleenp, we decided that placing the assert at the source of the nullptr would be a better indicate the problem should it arise in the code. The description has been updated with more detail.

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

PR Comment: https://git.openjdk.org/jdk/pull/16405#issuecomment-1787393136