Skip to content

Commit

Permalink
8292635: Run ArchivedEnumTest.java in jdk tier testing
Browse files Browse the repository at this point in the history
Reviewed-by: iklam, dholmes
  • Loading branch information
calvinccheung committed Jan 19, 2023
1 parent fc9f8ba commit 910dffe
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 5 deletions.
4 changes: 1 addition & 3 deletions src/hotspot/share/cds/cdsHeapVerifier.cpp
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2022, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2022, 2023, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -308,11 +308,9 @@ int CDSHeapVerifier::trace_to_root(outputStream* st, oop orig_obj, oop orig_fiel
return level;
}

#ifdef ASSERT
void CDSHeapVerifier::verify() {
CDSHeapVerifier verf;
HeapShared::archived_object_cache()->iterate(&verf);
}
#endif

#endif // INCLUDE_CDS_JAVA_HEAP
4 changes: 2 additions & 2 deletions src/hotspot/share/cds/cdsHeapVerifier.hpp
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2022, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2022, 2023, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -82,7 +82,7 @@ class CDSHeapVerifier : public KlassClosure {
// For ResourceHashtable::iterate()
inline bool do_entry(oop& orig_obj, HeapShared::CachedOopInfo& value);

static void verify() NOT_DEBUG_RETURN;
static void verify();

static void trace_to_root(outputStream* st, oop orig_obj);
};
Expand Down
1 change: 1 addition & 0 deletions test/jdk/TEST.ROOT
Expand Up @@ -60,6 +60,7 @@ requires.properties= \
vm.compiler1.enabled \
vm.compiler2.enabled \
vm.cds \
vm.cds.write.archived.java.heap \
vm.continuations \
vm.musl \
vm.debug \
Expand Down

3 comments on commit 910dffe

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

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

@JesperIRL
Copy link
Member

Choose a reason for hiding this comment

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

/tag jdk-21+6

@openjdk
Copy link

@openjdk openjdk bot commented on 910dffe Jan 19, 2023

Choose a reason for hiding this comment

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

@JesperIRL The tag jdk-21+6 was successfully created.

Please sign in to comment.