RFR: 8343148: C2: Refactor uses of "PhaseValue::*con*() + PhaseIdealLoop::set_ctrl()" into separate method [v8]

theoweidmannoracle duke at openjdk.org
Wed Nov 13 10:24:32 UTC 2024


> This patch introduces the methods `PhaseIdealLoop::intcon` and `PhaseIdealLoop::longcon` which are wrappers for:
> 
> 
> ConINode* node = _igvn.intcon(i);
> set_ctrl(node, C->root());
> 
> 
> and
> 
> 
> ConLNode* node = _igvn.longcon(i);
> set_ctrl(node, C->root());
> 
> 
> Occurrences of this pattern in loopnode.cpp were replaced with the appropriate call to the new methods.

theoweidmannoracle has updated the pull request incrementally with two additional commits since the last revision:

 - Update src/hotspot/share/opto/loopTransform.cpp
   
   Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
 - Update src/hotspot/share/opto/loopTransform.cpp
   
   Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21836/files
  - new: https://git.openjdk.org/jdk/pull/21836/files/8c51ec99..38c6f510

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21836&range=07
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21836&range=06-07

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/21836.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21836/head:pull/21836

PR: https://git.openjdk.org/jdk/pull/21836


More information about the hotspot-compiler-dev mailing list