RFR: 8318682: SA decoding of scalar replaced objects is broken [v7]

Tom Rodriguez never at openjdk.org
Tue Apr 30 17:36:39 UTC 2024


> The changes for JDK-8287061 didn't update the SA decoding logic and there are other places where the decoding has gotten out of sync with HotSpot.  Some of them can't be tested because they are part of JVMCI but I've added a directed test for the JDK-8287061 code and a more brute force test that tries to decode everything.

Tom Rodriguez 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 10 additional commits since the last revision:

 - Merge branch 'master' into tkr-hsdb-assert
 - Merge branch 'master' into tkr-hsdb-assert
 - Review comments
 - Remove testdebuginfodecode command
 - Update src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/NMethod.java
   
   Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>
 - Update src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/CommandProcessor.java
   
   Co-authored-by: Andrey Turbanov <turbanoff at gmail.com>
 - Pass the proper options to the lingered app
 - Fix problem list and correct jtreg arguments
 - Add missing files
 - 8318682: SA decoding of scalar replaced objects is broken

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17407/files
  - new: https://git.openjdk.org/jdk/pull/17407/files/f25c92ef..3cafefc6

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17407&range=06
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17407&range=05-06

  Stats: 663952 lines in 9058 files changed: 132224 ins; 168721 del; 363007 mod
  Patch: https://git.openjdk.org/jdk/pull/17407.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17407/head:pull/17407

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


More information about the serviceability-dev mailing list