Skip to content

Commit aeaa0fb

Browse files
committed
[JDK-8364586] Adapt JDK-8364187: Make getClassAccessFlagsRaw non-native
PullRequest: graal/21749
2 parents 9d937d3 + 06a145b commit aeaa0fb

File tree

2 files changed

+3
-17
lines changed

2 files changed

+3
-17
lines changed

common.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
"COMMENT.jdks": "When adding or removing JDKs keep in sync with JDKs in ci/common.jsonnet",
1010
"jdks": {
11-
"galahad-jdk": {"name": "jpg-jdk", "version": "26", "build_id": "jdk-26+9-846", "platformspecific": true, "extrabundles": ["static-libs"]},
11+
"galahad-jdk": {"name": "jpg-jdk", "version": "26", "build_id": "jdk-26+10-966", "platformspecific": true, "extrabundles": ["static-libs"]},
1212

1313
"oraclejdk17": {"name": "jpg-jdk", "version": "17.0.7", "build_id": "jdk-17.0.7+8", "platformspecific": true, "extrabundles": ["static-libs"]},
1414
"labsjdk-ce-17": {"name": "labsjdk", "version": "ce-17.0.7+4-jvmci-23.1-b02", "platformspecific": true },

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/hotspot/meta/HotSpotGraphBuilderPlugins.java

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ public void run() {
289289
registerMathPlugins(invocationPlugins, target.arch);
290290
registerContinuationPlugins(invocationPlugins, config);
291291
registerCallSitePlugins(invocationPlugins);
292-
registerReflectionPlugins(invocationPlugins, config);
292+
registerReflectionPlugins(invocationPlugins);
293293
registerAESPlugins(invocationPlugins, config);
294294
registerAdler32Plugins(invocationPlugins, config);
295295
registerCRC32Plugins(invocationPlugins, config);
@@ -508,7 +508,7 @@ public boolean apply(GraphBuilderContext b, ResolvedJavaMethod targetMethod, Rec
508508
plugins.register(VolatileCallSite.class, plugin);
509509
}
510510

511-
private static void registerReflectionPlugins(InvocationPlugins plugins, GraalHotSpotVMConfig config) {
511+
private static void registerReflectionPlugins(InvocationPlugins plugins) {
512512
Registration r = new Registration(plugins, "jdk.internal.reflect.Reflection");
513513
r.register(new InlineOnlyInvocationPlugin("getCallerClass") {
514514
@Override
@@ -517,20 +517,6 @@ public boolean apply(GraphBuilderContext b, ResolvedJavaMethod targetMethod, Rec
517517
return true;
518518
}
519519
});
520-
r.register(new InvocationPlugin("getClassAccessFlags", Class.class) {
521-
@Override
522-
public boolean apply(GraphBuilderContext b, ResolvedJavaMethod targetMethod, Receiver receiver, ValueNode arg) {
523-
try (HotSpotInvocationPluginHelper helper = new HotSpotInvocationPluginHelper(b, targetMethod, config)) {
524-
ValueNode klass = helper.readKlassFromClass(b.nullCheckedValue(arg));
525-
// Primitive Class case
526-
ValueNode klassNonNull = helper.emitNullReturnGuard(klass, ConstantNode.forInt(Modifier.ABSTRACT | Modifier.FINAL | Modifier.PUBLIC), GraalDirectives.UNLIKELY_PROBABILITY);
527-
// Return (Klass::_access_flags & jvmAccWrittenFlags)
528-
ValueNode accessFlags = helper.readKlassAccessFlags(klassNonNull);
529-
helper.emitFinalReturn(JavaKind.Int, accessFlags);
530-
}
531-
return true;
532-
}
533-
});
534520
}
535521

536522
private static final SpeculationReasonGroup JVMTI_NOTIFY_ALLOCATE_INSTANCE = new SpeculationReasonGroup("JvmtiNotifyAllocateInstance");

0 commit comments

Comments
 (0)