RFR: 8261268: LOAD_INSTANCE placeholders unneeded for parallelCapable class loaders

Coleen Phillimore coleenp at openjdk.java.net
Tue Feb 9 15:06:48 UTC 2021


On Tue, 9 Feb 2021 02:56:46 GMT, David Holmes <dholmes at openjdk.org> wrote:

>The actual logic changes seem sound (though some of the reorganisation was a little hard to follow).

Unfortunately the diffs are bad because I did two things.  I changed the indentation and I check for the class in the dictionary before adding the LOAD_INSTANCE placeholder for the !parallelCapable() && !bootclass case.

> I have some comments on comments, but overall this seems okay provided all testing passes.
Yup, thanks!

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

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


More information about the hotspot-runtime-dev mailing list