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