C2: No need to allocate value types for uncommon trap's debug info

Roland Westrelin rwestrel at redhat.com
Wed Dec 14 08:13:27 UTC 2016


Hi Tobias,

>> http://cr.openjdk.java.net/~roland/valhalla/noallocatunc/webrev.00/
>
> Why is that code not needed anymore?

  case Op_ValueType: {
    ValueTypeNode* vt = n->as_ValueType();
    vt->make_scalar_in_safepoints(this);
    if (vt->outcnt() == 0) {
      vt->disconnect_inputs(NULL, this);
    }
    break;
  }

in Compile::final_graph_reshaping() should change the debug info of the
uncommon traps so it doesn't include any value types or am I missing
something?

Roland.


More information about the valhalla-dev mailing list