RFR: JDK-8279956: Useless method Scheduling::ComputeLocalLatenciesForward() [v3]

SUN Guoyun duke at openjdk.java.net
Tue Jan 18 06:18:30 UTC 2022


On Fri, 14 Jan 2022 08:25:06 GMT, SUN Guoyun <duke at openjdk.java.net> wrote:

>> Method `Scheduling::ComputeLocalLatenciesForward()` be used for compute the latency of all the instructions and write the result to `_node_latency`. But I found that `_node_latency` was never used. So I think method `Scheduling::ComputeLocalLatenciesForward()` can be deleted, which is also an enhancement to the efficiency of compilation threads.
>
> SUN Guoyun has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   8279956: Useless method Scheduling::ComputeLocalLatenciesForward( )

Test result on x86 used command `make test TEST=":tier1 :tier2 :tier3" CONF=fast JTREG="VM_OPTIONS=-XX:+OptoScheduling;TIMEOUT_FACTOR=10"` 

<pre>
==============================
Test summary
==============================
   TEST                                              TOTAL  PASS  FAIL ERROR   
   jtreg:test/hotspot/jtreg:tier1                     1765  1765     0     0   
>> jtreg:test/jdk:tier1                               2072  2071     1     0 <<
   jtreg:test/langtools:tier1                         4243  4243     0     0   
   jtreg:test/jaxp:tier1                                 0     0     0     0   
   jtreg:test/lib-test:tier1                             0     0     0     0   
   jtreg:test/hotspot/jtreg:tier2                      441   441     0     0   
>> jtreg:test/jdk:tier2                               3784  3770    10     4 <<
   jtreg:test/langtools:tier2                           11    11     0     0   
   jtreg:test/jaxp:tier2                               452   452     0     0   
   jtreg:test/hotspot/jtreg:tier3                       72    72     0     0   
   jtreg:test/jdk:tier3                                174   174     0     0   
   jtreg:test/langtools:tier3                            0     0     0     0   
   jtreg:test/jaxp:tier3                                 0     0     0     0   
==============================
TEST FAILURE
</pre>

 The test FAIL and ERROR items has nothing to do with this patch.

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

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


More information about the hotspot-compiler-dev mailing list