Skip to content

Commit

Permalink
8260933: runtime/cds/serviceability/ReplaceCriticalClassesForSubgraph…
Browse files Browse the repository at this point in the history
…s.java fails without CompactStrings

Reviewed-by: iklam
  • Loading branch information
jianglizhou committed Mar 2, 2021
1 parent d339832 commit 9f0f0c9
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -42,12 +42,12 @@ public String[] getTests() {
String tests[] = {
// Try to replace classes that are used by the archived subgraph graphs. (CDS should be disabled)
"-early -notshared -subgraph java/lang/module/ResolvedModule jdk.internal.module.ArchivedModuleGraph",
"-early -notshared -subgraph java/lang/Long java.lang.Long$LongCache",
"-early -notshared -subgraph java/lang/Integer java.lang.Integer$IntegerCache",

// CDS should not be disabled -- these critical classes cannot be replaced because
// JvmtiExport::early_class_hook_env() is false.
"-subgraph java/lang/module/ResolvedModule jdk.internal.module.ArchivedModuleGraph",
"-subgraph java/lang/Long java.lang.Long$LongCache",
"-subgraph java/lang/Integer java.lang.Integer$IntegerCache",

// Tests for archived full module graph. We cannot use whitebox, which requires appending to bootclasspath.
// VM will disable full module graph if bootclasspath is appended.
Expand Down

1 comment on commit 9f0f0c9

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.