Integrated: 8356328: Some C2 IR nodes miss size_of() function

kuaiwei duke at openjdk.org
Sun May 18 23:30:55 UTC 2025


On Wed, 7 May 2025 07:04:26 GMT, kuaiwei <duke at openjdk.org> wrote:

> I wrote a test to check if every C2 IR node has correct size_of() function. And I found some of them are missed. They added new fields and not add size_of() to reflect new size. In linux, it does not cause issue so far, because gcc allocate more space for alignment and can keep these additional `bool` flags. But it will report failure on windows. And if anyone modified base class, it will cause problem.
> 
> PS, My test is in https://github.com/openjdk/jdk/compare/master...kuaiwei:jdk:test/check_node_size , but it has many hack on IR nodes to make test to run.

This pull request has now been integrated.

Changeset: 9927ec0b
Author:    Kuai Wei <kuaiwei.kw at alibaba-inc.com>
Committer: Shaojin Wen <swen at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/9927ec0b91775db342b2bbc1937253325c367a19
Stats:     28 lines in 4 files changed: 20 ins; 4 del; 4 mod

8356328: Some C2 IR nodes miss size_of() function

Reviewed-by: thartmann, chagedorn

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

PR: https://git.openjdk.org/jdk/pull/25081


More information about the hotspot-compiler-dev mailing list