archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-08-28 15:24:43 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8241320: The ClassLoaderData::_is_unsafe_anonymous field is unused in the SA

Browse source 
Remove unused code that is changing in Hotspot for hidden classes.

Reviewed-by: lfoltan, dholmes, sspitsyn
This commit is contained in:
Coleen Phillimore 2020-03-20 07:30:52 -04:00
parent 95d2830361
commit 6dffcf7533
2 changed files with 1 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

1
src/hotspot/share/runtime/vmStructs.cpp
View file

@ -522,7 +522,6 @@ typedef HashtableEntry<InstanceKlass*, mtClass> KlassHashtableEntry;
nonstatic_field(ClassLoaderData, _class_loader, OopHandle) \ nonstatic_field(ClassLoaderData, _class_loader, OopHandle) \
nonstatic_field(ClassLoaderData, _next, ClassLoaderData*) \ nonstatic_field(ClassLoaderData, _next, ClassLoaderData*) \
volatile_nonstatic_field(ClassLoaderData, _klasses, Klass*) \ volatile_nonstatic_field(ClassLoaderData, _klasses, Klass*) \
nonstatic_field(ClassLoaderData, _is_unsafe_anonymous, bool) \
volatile_nonstatic_field(ClassLoaderData, _dictionary, Dictionary*) \ volatile_nonstatic_field(ClassLoaderData, _dictionary, Dictionary*) \
\ \
static_ptr_volatile_field(ClassLoaderDataGraph, _head, ClassLoaderData*) \ static_ptr_volatile_field(ClassLoaderDataGraph, _head, ClassLoaderData*) \

8
src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/classfile/ClassLoaderData.java
View file

@ -1,5 +1,5 @@
/* /*
* Copyright (c) 2012, 2019, Oracle and/or its affiliates. All rights reserved. * Copyright (c) 2012, 2020, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
* *
* This code is free software; you can redistribute it and/or modify it * This code is free software; you can redistribute it and/or modify it
@ -44,14 +44,12 @@ public class ClassLoaderData extends VMObject {
classLoaderFieldOffset = type.getAddressField("_class_loader").getOffset(); classLoaderFieldOffset = type.getAddressField("_class_loader").getOffset();
nextField = type.getAddressField("_next"); nextField = type.getAddressField("_next");
klassesField = new MetadataField(type.getAddressField("_klasses"), 0); klassesField = new MetadataField(type.getAddressField("_klasses"), 0);
isUnsafeAnonymousField = new CIntField(type.getCIntegerField("_is_unsafe_anonymous"), 0);
dictionaryField = type.getAddressField("_dictionary"); dictionaryField = type.getAddressField("_dictionary");
} }
private static long classLoaderFieldOffset; private static long classLoaderFieldOffset;
private static AddressField nextField; private static AddressField nextField;
private static MetadataField klassesField; private static MetadataField klassesField;
private static CIntField isUnsafeAnonymousField;
private static AddressField dictionaryField; private static AddressField dictionaryField;
public ClassLoaderData(Address addr) { public ClassLoaderData(Address addr) {
@ -76,10 +74,6 @@ public class ClassLoaderData extends VMObject {
return vmOopHandle.resolve(); return vmOopHandle.resolve();
} }
public boolean getisUnsafeAnonymous() {
return isUnsafeAnonymousField.getValue(this) != 0;
}
public ClassLoaderData next() { public ClassLoaderData next() {
return instantiateWrapperFor(nextField.getValue(getAddress())); return instantiateWrapperFor(nextField.getValue(getAddress()));
} }