[jdk8u-dev] Integrated: 8233364: Fix undefined behavior in Canonicalizer::do_ShiftOp

[Martin Balao]

On Thu, 12 Sep 2024 04:24:35 GMT, Martin Balao <mbalao at openjdk.org> wrote:

> Hi,
> 
> I'd like to propose an 8u backport of 8233364.
> 
> The 11u backport of 8233364 does not apply cleanly or requires changes because of the following reasons:
> 
>   * Context differences in the change due to the absence of 8182299 in 8u.
> 
>   * 8181449 is not in 8u, so an include of "utilities/debug.hpp" would need to be added for STATIC_ASSERT to be available in utilities/globalDefinitions.hpp. However, this generates a dependency issue and we would need to backport the 8181449 enhancement anyways. 8181449 is a big change and would be too risky for 8u. I added the assertion using the old style.
> 
>   * 8u does not have 8148244 (gtest).
> 
> All tests in the hotspot/test/compiler/c1 category passed.
> 
> Thanks,
> Martin.-

This pull request has now been integrated.

Changeset: 73056eb8
Author:    Martin Balao <mbalao at openjdk.org>
URL:       https://git.openjdk.org/jdk8u-dev/commit/73056eb8d4c76ce38d06acb25628ff0d61e182c3
Stats:     50 lines in 2 files changed: 36 ins; 4 del; 10 mod

8233364: Fix undefined behavior in Canonicalizer::do_ShiftOp

Add java_shift_xxx helpers and use them.

Reviewed-by: sgehwolf, roland
Backport-of: 89e3782f218a54c91cbab9f06e75825b90dd2fb7

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

PR: https://git.openjdk.org/jdk8u-dev/pull/577