RFR: 8286104: use aggressive liveness for unstable_if traps [v7]

Xin Liu xliu at openjdk.java.net
Wed Jun 1 19:06:18 UTC 2022


> I found that some phi nodes are useful because its uses are uncommon_trap nodes. In worse case, it would hinder boxing object eliminating and scalar replacement. Test.java of JDK-8286104 reveals this issue. This patch allows c2 parser to collect liveness based on next bci for unstable_if traps.  In most cases, next bci is closer to exits, so live locals are diminishing.  It helps to reduce the number of inputs of unstable_if traps. 
> 
> This is not a REDO of Optimization of Box nodes in uncommon_trap(JDK-8261137). Two patches are orthogonal.  I adapt test from [TestEliminateBoxInDebugInfo.java](https://github.com/openjdk/jdk/pull/2401/files#diff-49b2e38825aa4c28ca196bdc70c3cbecc2e835c2899f4f393527df4796b177ea), so part of credit goes to the original author.  I found that Scalar replacement can take care of the object `Integer ii = Integer.valueOf(value)` in original test even it can't be removed by later inliner.  I tweak the profiling data of Integer.valueOf() to hinder scalar replacement. 
> 
> This patch can cover the problem discovered by JDK-8261137. I ran the microbench and got 9x speedup on x86_64. It's almost same as JDK-8261137. 
> 
> Before:  
> 
> Benchmark               Mode  Cnt   Score   Error  Units
> MyBenchmark.testMethod  avgt   10  32.776 ± 0.075  us/op
> 
> After: 
> 
> Benchmark               Mode  Cnt   Score   Error  Units
> MyBenchmark.testMethod  avgt   10  3.656 ± 0.006  us/op
>  ```
> 
> Testing
> I ran tier1 test with and without `-XX:+DeoptimizeALot`. No regression has been found yet.

Xin Liu has updated the pull request incrementally with one additional commit since the last revision:

  Remove useless flag. if jdwp is on, liveness_at_bci() marks all local
  variables live.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8545/files
  - new: https://git.openjdk.java.net/jdk/pull/8545/files/4fdd1c88..6e9f2670

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8545&range=06
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8545&range=05-06

  Stats: 26 lines in 7 files changed: 1 ins; 15 del; 10 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8545.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8545/head:pull/8545

PR: https://git.openjdk.java.net/jdk/pull/8545


More information about the hotspot-compiler-dev mailing list