Integrated: 8302145: ddepth should be uint in PhaseIdealLoop::register_node()

Eric Nothum duke at openjdk.org
Wed Jun 7 11:43:07 UTC 2023


On Tue, 6 Jun 2023 12:50:50 GMT, Eric Nothum <duke at openjdk.org> wrote:

> Changed the type of the ddepth parameter in PhaseIdealLoop::register_node from int to uint, to avoid potentially unsafe uint -> int casts.
> The method is often used with uint arguments, coming from dom_depth().
> I have verified the calls of PhaseIdealLoop::register_node and have removed related casts.

This pull request has now been integrated.

Changeset: 1de40f36
Author:    Eric Nothum <eric.nothum at oracle.com>
Committer: Christian Hagedorn <chagedorn at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/1de40f360f3beed5eb9fbd62a992989bb5bdb315
Stats:     4 lines in 2 files changed: 0 ins; 0 del; 4 mod

8302145: ddepth should be uint in PhaseIdealLoop::register_node()

Reviewed-by: chagedorn, thartmann

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

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


More information about the hotspot-compiler-dev mailing list