RFR: 8311591: Add SystemModulesPlugin test case that splits module descriptors with new local variables defined by DedupSetBuilder [v3]

Christoph duke at openjdk.org
Sun Aug 13 16:43:58 UTC 2023


> Add new test case with sample modules that contains some requires/exports/uses/provides.
> 
> We are just unsure if and how we should add some last step of verificaiton with the extracted and decompiled class.
> 
> Follow up task from https://github.com/openjdk/jdk/pull/14408

Christoph 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 15 additional commits since the last revision:

 - Merge remote-tracking branch 'origin/fix-8311591' into fix-8311591
   
   * origin/fix-8311591:
     8311591: Add SystemModulesPlugin test case that splits module descriptors with new local variables defined by DedupSetBuilder
 - 8311591: Add SystemModulesPlugin test case that splits module descriptors with new local variables defined by DedupSetBuilder
   
   Co-authored-by: Oliver Kopp <kopp.dev at gmail.com>
 - Merge remote-tracking branch 'upstream/master' into fix-8311591
   
   * upstream/master: (49 commits)
     8313904: [macos] All signing tests which verifies unsigned app images are failing
     8314139: TEST_BUG: runtime/os/THPsInThreadStackPreventionTest.java could fail on machine with large number of cores
     8313798: [aarch64] sun/tools/jhsdb/HeapDumpTestWithActiveProcess.java sometimes times out on aarch64
     8313244: NM flags handling in configure process
     8314113: G1: Remove unused G1CardSetInlinePtr::card_at
     8311648: Refactor the Arena/Chunk/ChunkPool interface
     8313224: Avoid calling JavaThread::current() in MemAllocator::Allocation constructor
     8312461: JNI warnings in SunMSCApi provider
     8312882: Update the CONTRIBUTING.md with pointers to lifecycle of a PR
     8304292: Memory leak related to ClassLoader::update_class_path_entry_list
     8313899: JVMCI exception Translation can fail in TranslatedException.<clinit>
     8313633: [macOS] java/awt/dnd/NextDropActionTest/NextDropActionTest.java fails with java.lang.RuntimeException: wrong next drop action!
     8312259: StatusResponseManager unused code clean up
     8314061: [JVMCI] DeoptimizeALot stress logic breaks deferred barriers
     8313905: Checked_cast assert in CDS compare_by_loader
     8313654: Test WaitNotifySuspendedVThreadTest.java timed out
     8312194: test/hotspot/jtreg/applications/ctw/modules/jdk_crypto_ec.java cannot handle empty modules
     8313616: support loading library members on AIX in os::dll_load
     8313882: Fix -Wconversion warnings in runtime code
     8313239: InetAddress.getCanonicalHostName may return ip address if reverse lookup fails
     ...
 - cleanup
 - rename and cleanup
 - revert back to default size of 75 for module desriptors
   use parameter for jlink
 - add decompilation via javap
 - extract jimage
 - add some more opens and exports
   
   prepare methods for verifying
 - Merge remote-tracking branch 'origin/fix-8311591' into fix-8311591
   
   * origin/fix-8311591:
     Set module split size to 1
 - ... and 5 more: https://git.openjdk.org/jdk/compare/406ae9ec...996c187c

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15234/files
  - new: https://git.openjdk.org/jdk/pull/15234/files/9b9b0bb2..996c187c

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

  Stats: 334 lines in 30 files changed: 137 ins; 78 del; 119 mod
  Patch: https://git.openjdk.org/jdk/pull/15234.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15234/head:pull/15234

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


More information about the core-libs-dev mailing list