RFR: 8308771: Redundant header file in interp_masm_<archs>.hpp

Amit Kumar amitkumar at openjdk.org
Thu May 25 03:13:08 UTC 2023


`asm/macroAssembler` and `interpreter/invocationCounter.hpp` can be sorted out in `interp_masm_<arch>.hpp` files. I have tested these changes for `s390x` and `arch64-M1 Pro` for `release` and `fastdebug` build. Builds are good. 

For other archs(`arm`, `risc-v`, `ppc`) I haven't performed any testing because I don't have access to such hardwares. 

I've restored the changes for x86 as it seemed to break the build, not sure why.

__Reviews and comments are welcomed__

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

Commit messages:
 - restore x86
 - cleanup 2
 - clean-up 1

Changes: https://git.openjdk.org/jdk/pull/14137/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14137&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8308771
  Stats: 21 lines in 10 files changed: 1 ins; 18 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/14137.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14137/head:pull/14137

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


More information about the hotspot-dev mailing list