RFR: 8292059: Do not inline InstanceKlass::allocate_instance() [v7]

Afshin Zafari duke at openjdk.org
Wed Mar 15 13:46:42 UTC 2023


On Wed, 15 Mar 2023 12:51:53 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Good.  I think devirtualizer.inline.hpp is used by the GC inlined functions so should remain here.
>
> Sorry for being confusing before.  My comment was that you should remove #includes that aren't directly referenced by this .inline.hpp file.  I see that you've done this now, but do you need classfile/vmSymbols.hpp here?  I don't see it.

I mis-interpreted the required changes.
It is not needed here, and moved to `jvmtiEnvBase.cpp`.

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

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


More information about the hotspot-dev mailing list