RFR: 8183137: [MVT] C2 doesn't eliminate redundant value type allocations inside loops

Tobias Hartmann tobias.hartmann at oracle.com
Thu Jul 13 11:04:14 UTC 2017

Hi Roland,

> http://cr.openjdk.java.net/~roland/8183137/webrev.01/

This looks good to me!

Just some minor things:
- little typo in valuetypenode.hpp:134  ValueTypeNode -> ValueTypePtrNode
- for better readability, you could merge test95_mt/test95_mt3/test96_mt/test96_mt3 in ValueTypeTestBench.java because they are identical

> There are tests such as test95/test96 for which the allocation is still
> not eliminated but could be by improving this further.

Is this because C2 does not inline the non-static method handle calls? You've mentioned off-thread that you are working on fixing this. What's your plan there?


More information about the valhalla-dev mailing list