RFR: 8255617: Zero: purge the remaining bytecode interpreter profiling support [v2]

Aleksey Shipilev shade at openjdk.java.net
Tue Nov 3 10:22:14 UTC 2020


> All the stubs in `interpreter/zero/bytecodeInterpreterProfiling.hpp` are empty. History shows the whole thing gradually moved to template interpreter. We can probably simplify Zero code by dropping these empty stubs altogether. Arguably, this makes porting to new architectures a bit harder, but it seems that the proper stepping stone after Zero is implementing template interpreter anyway.
> 
> On my TR 3970X, this improves:
>  - Linux x86_64 Zero "make images" times from ~18 minutes to ~17.5 minutes
> 
> I would like to have the opinion of @GoeLin who added this for PPC64 porting back in 8u. And probably @DamonFool who is usually interested in Zero. And @jerboaa, @gnu-andrew who deal with Zero from time to time.

Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:

 - Merge branch 'master' into JDK-8255617-zero-purge-bi-profiliing
 - Also remove now unused BytecodeInterpreter::mdx field
 - Remove leftover variable
 - Remove leftover comments
 - 8255617: Zero: purge the remaining bytecode interpreter profiling support

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/944/files
  - new: https://git.openjdk.java.net/jdk/pull/944/files/2bcc4e31..15df2843

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=944&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=944&range=00-01

  Stats: 5456 lines in 267 files changed: 3019 ins; 1050 del; 1387 mod
  Patch: https://git.openjdk.java.net/jdk/pull/944.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/944/head:pull/944

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


More information about the hotspot-dev mailing list