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