RFR(M): 8200303: C2 should leverage profiling for lookupswitch/tableswitch

Roland Westrelin rwestrel at redhat.com
Thu Apr 5 09:54:16 UTC 2018


Hi Tobias,

Here is a new webrev:

http://cr.openjdk.java.net/~roland/8200303/webrev.02/

I fixed the typo and added MachJump to vmstructs.

I don't understand how the MachIfNode class in the SA is supposed to be
used. I tried dumping a graph that has a MachIfNode from clhsdb and the
MachIfNode shows up as unknown:

 7      UnknownNode<0x00007fe79787c428> ===  9  8    [[ 6  28 ]]
 28     IfFalse ===  7  [[ 27 ]]

So I'm not sure it's worth the trouble of adding a MachJumpNode.

Roland.


More information about the hotspot-compiler-dev mailing list