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

theoweidmannoracle duke at openjdk.org
Thu Nov 7 16:11:07 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 one additional commit since the last revision:

  Update src/hotspot/share/opto/loopopts.cpp
  
  Co-authored-by: Emanuel Peter <emanuel.peter at oracle.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21836/files
  - new: https://git.openjdk.org/jdk/pull/21836/files/d1817ee8..798a6172

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

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 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