RFR: 8299155: C2: SubTypeCheckNode::verify() should not produce dependencies / oop pool entries

Tobias Hartmann thartmann at openjdk.org
Thu Feb 2 14:16:40 UTC 2023


On Thu, 2 Feb 2023 09:07:39 GMT, Roland Westrelin <roland at openjdk.org> wrote:

> This patch simply guarantees a new LoadKlassNode is not created (and
> processed by igvn) unless it's really needed.

src/hotspot/share/opto/subtypenode.cpp line 206:

> 204: }
> 205: 
> 206: Node* SubTypeCheckNode::load_klass(PhaseGVN* phase, Node* obj_or_subklass) const {

I think this should either be a static method or the `obj_or_subklass` argument should be removed and replaced by `in(ObjOrSubKlass)` in the method body.

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

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


More information about the hotspot-compiler-dev mailing list