JDK-8366361 : C2 SuperWord: rename VTransformNode::set_req -> init_req, analogue to Node::init_req
  • Type: Sub-task
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 26
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2025-08-28
  • Updated: 2025-09-02
  • Resolved: 2025-09-01
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 26
26 masterFixed
Description
I'm working on cost-modeling, and am integrating some smaller changes from this proof-of-concept PR:
https://github.com/openjdk/jdk/pull/20964

This is a pure refactoring - no change in behaviour. I'm presenting it like this because it will make reviews easier.

The current implementation of VTransformNode::set_req has init_req semantics, it verifies that the corresponding input is still nullptr. We should thus rename it. It will also free up the "set_req" name for later use in VTransform optimizations, where we want to modify the graph.

See VTransformReductionVectorNode::optimize_move_non_strict_order_reductions_out_of_loop in the proof-of-concept PR.
Comments
Changeset: 56713817 Branch: master Author: Emanuel Peter <epeter@openjdk.org> Date: 2025-09-01 08:47:19 +0000 URL: https://git.openjdk.org/jdk/commit/56713817c0fd060f7106a538b0e795081f4f9d4b
01-09-2025

sh jib.sh mach5 -- remote-build-and-test --email emanuel.peter@oracle.com --slack-channel @empeter --job tier1 --jvm-args "" --comment "JDK-8366361-vtn-init_req v5" --patch-based https://mach5.us.oracle.com/mdash/jobs/empeter-jdk-fork6-20250901-0711-33700112
01-09-2025

A pull request was submitted for review. Branch: master URL: https://git.openjdk.org/jdk/pull/26991 Date: 2025-08-28 15:30:31 +0000
01-09-2025