JDK-8335552 : [8u] JDK-8303466 backport to 8u requires 3 ::Identity signature fixes
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: openjdk8u432
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2024-07-02
  • Updated: 2024-07-27
  • Resolved: 2024-07-03
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
openjdk8u422Fixed
Related Reports
Relates :  
JDK-8303466 - C2: failed: malformed control flow. Limit type made precise with MaxL/MinL
Description
As noted by [~rmarchenko] [1], the 8u backport of JDK-8303466, JDK-8335472, has been integrated with incorrect signatures for MaxLNode::Identity, MinLNode::Identity and ConvI2LNode::Identity.

This would have been caught by GitHub Actions in 11u, 17u and 21u repositories that have JDK-8152666, where the -Woverloaded-virtual [2] GCC compile flag was added. NOTE: warnings are treated as errors by default, and GitHub Actions execute configure without passing --disable-warnings-as-errors.

[1] https://github.com/openjdk/jdk8u-dev/pull/529#discussion_r1661486052
[2] https://gcc.gnu.org/onlinedocs/gcc-14.1.0/gcc/C_002b_002b-Dialect-Options.html#index-Wno-overloaded-virtual
Comments
Changeset: a717515a Branch: master Author: Francisco Ferrari Bihurriet <fferrari@openjdk.org> Committer: Andrew John Hughes <andrew@openjdk.org> Date: 2024-07-03 23:36:15 +0000 URL: https://git.openjdk.org/jdk8u-dev/commit/a717515a4e561caa1560fae19e7a30ca743da4b0
27-07-2024
A pull request was submitted for review. Branch: master URL: https://git.openjdk.org/jdk8u-dev/pull/532 Date: 2024-07-02 14:23:58 +0000
09-07-2024
Changeset: 16ea85b5 Author: Francisco Ferrari Bihurriet <fferrari@openjdk.org> Committer: Martin Balao <mbalao@openjdk.org> Date: 2024-07-03 23:36:15 +0000 URL: https://git.openjdk.org/jdk8u-dev/commit/16ea85b54115073c6c22377b817878fe112b2100
03-07-2024
[jdk8u-fix-request] Approval Request from Francisco Ferrari Bihurriet jdk8u is affected by this bug after the JDK-8335472 integration. ::Identity methods are not being invoked, leading to sub-optimal code for MaxLNode, MinLNode and ConvI2LNode. The patch has been reviewed and GitHub Actions passed (except for macOS, where the jobs are still waiting for an available runner to pick them).
03-07-2024