RFR(M): 8231561: [lworld] C2 generates inefficient code for acmp
tobias.hartmann at oracle.com
Thu Oct 10 13:00:12 UTC 2019
thanks for looking at this!
On 10.10.19 13:17, Roland Westrelin wrote:
> Don't use need to also change the cmp() implementation so it takes the
> _clear_prop_bits into account? Otherwise you would risk replacing a
> LoadKlass with _clear_prop_bits = true with one with _clear_prop_bits =
> false which could cause a crash. I suppose replacing one with
> _clear_prop_bits = false with one with _clear_prop_bits = true is ok
> though but too complicated to handle.
Good catch, I've added hash/cmp implementations to LoadKlassNode and LoadNKlassNode:
More information about the valhalla-dev