RFR(S): 8238869: [lworld] C2's slow path allocation does not set the larval bit

Tobias Hartmann tobias.hartmann at oracle.com
Fri Feb 14 14:53:38 UTC 2020


On 14.02.20 12:42, Tobias Hartmann wrote:
> Right, it probably doesn't make a difference. Here's a new webrev:
> http://cr.openjdk.java.net/~thartmann/8238869/webrev.01/

Just noticed that with this, we need to update our IR match rules to handle the initialization of
the new argument. While fixing that, I've noticed that the match rule for array allocation is broken
(doesn't handle "Q" in the precise klass statement). I've fixed it as well and added some additional
annotations to the tests to catch this in the future.

Updated webrev in-place.

Thanks,
Tobias



More information about the valhalla-dev mailing list