RFR: 8215477: [lworld] C2 should avoid value type allocations when inlining incrementally

Roland Westrelin rwestrel at redhat.com
Fri Feb 15 12:48:38 UTC 2019


Hi Tobias,

Thanks for looking at this.

> I think you can use igvn.remove_dead_node(n) in compile.cpp:2082.

Right. But is igvn.remove_dead_node(n) better/different than pushing on
igvn worklist?

> Just wondering, why not always create a ValueTypePtrNode here?

I can't see when that would help. Can you think of a scenario where
having a ValueTypePtrNode as an input to a non inlined call would help?

Roland.


More information about the valhalla-dev mailing list