RFR: 8356328: Some C2 IR nodes miss size_of() function [v2]

Fri May 9 06:43:52 UTC 2025

On Fri, 9 May 2025 06:05:31 GMT, kuaiwei <duke at openjdk.org> wrote:

>> Thanks for doing that but it's not only nice to have, right? GVN might otherwise incorrectly common two different nodes.
>
> I found `OpaqueNode` can not have `cmp` and `hash`, it will fail some tests. I think these nodes can not be optimized by GVN.

`OpaqueMultiversioningNode` inherits from `Opaque1` which defines `NO_HASH`.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25081#discussion_r2081034451