JDK-8330106 : C2: VectorInsertNode::make() shouldn't call ConINode::make() directly
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 16,17,21,23
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2024-04-11
  • Updated: 2025-04-07
  • Resolved: 2024-04-12
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.
JDK 21 JDK 23
21.0.8Fixed 23 b19Fixed
Related Reports
Causes :  
Description
It should call PhaseValues::intcon() instead.
Comments
A pull request was submitted for review. Branch: master URL: https://git.openjdk.org/jdk17u-dev/pull/3433 Date: 2025-04-03 17:31:27 +0000
03-04-2025

[jdk17u-fix-request] Approval Request from Satyen Subramaniam for backport. Adjusts code to call PhaseValues::intcon(), which is required so the node is properly entered in the GVN's hash table and its type is properly recorded in the GVN's type table Risk: Medium. Change to compiler code which can have broader impact. Change was merged in April 2024 and present in 23, which mitigates risk. Testing: * GHA Sanity Checks * Tier 1 and 2 tests locally
03-04-2025

[jdk21u-fix-request] Approval Request from Satyen Subramaniam for backport. Adjusts code to call PhaseValues::intcon(), which is required so the node is properly entered in the GVN's hash table and its type is properly recorded in the GVN's type table Risk: Low. Tip change was merged in April 2024 and change preserves functionality, addressing an uncommon edgecase. Testing: * GHA Sanity Checks * Tier 1 and 2 tests locally
03-04-2025

A pull request was submitted for review. Branch: master URL: https://git.openjdk.org/jdk21u-dev/pull/1579 Date: 2025-04-02 17:47:47 +0000
02-04-2025

Changeset: bde3fc0c Author: Roland Westrelin <roland@openjdk.org> Date: 2024-04-12 07:17:27 +0000 URL: https://git.openjdk.org/jdk/commit/bde3fc0c03c87d1f2605ae6bb84c33fadb7aa865
12-04-2024

ILW = Node not processed by IGVN in C2 (potential assert / missed optimization opportunity), never observed in current code, no workaround = MLH = P4
12-04-2024

A pull request was submitted for review. URL: https://git.openjdk.org/jdk/pull/18742 Date: 2024-04-11 13:51:29 +0000
11-04-2024