RFR: 8302152: Speed up compiler/loopopts/TestInfLoopNearUsePlacement.java [v2]

Emanuel Peter epeter at openjdk.org
Mon Feb 13 12:19:56 UTC 2023


> From over `5 sec` to about `0.8 sec`.
> 
> I simply reduced the `Thread.sleep`.
> 
> I verified that the same compilations still happen.
> 
> ~/Documents/jtreg/bin/jtreg -va -s -jdk:/home/emanuel/Documents/fork5-jdk/build/linux-x64-debug/jdk/ -javaoptions:"-XX:CompileCommand=printcompilation,compiler.loopopts.TestInfLoopNearUsePlacement::test" /home/emanuel/Documents/fork5-jdk/open/test/hotspot/jtreg/compiler/loopopts/TestInfLoopNearUsePlacement.java
> 
>     283   32 % !   3       compiler.loopopts.TestInfLoopNearUsePlacement::test @ 39 (61 bytes)
>     284   33 % !   4       compiler.loopopts.TestInfLoopNearUsePlacement::test @ 39 (61 bytes)
>     286   34   !   4       compiler.loopopts.TestInfLoopNearUsePlacement::test (61 bytes)
> 
> 
> On the time measurement that I got initially reported, this test was flagged as especially long-running, with `17 sec`. There may well be another issue, but I hope this fix already does the trick. We will see this in a while once I get the new data.

Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:

  add 4 other tests with infinite loops

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12514/files
  - new: https://git.openjdk.org/jdk/pull/12514/files/bf61a7a8..9a904a33

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

  Stats: 4 lines in 4 files changed: 0 ins; 0 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/12514.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12514/head:pull/12514

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


More information about the hotspot-compiler-dev mailing list