Skip to content

Commit

Permalink
8284909: [JVMCI] remove remnants of AOT support
Browse files Browse the repository at this point in the history
Reviewed-by: kvn
  • Loading branch information
Doug Simon committed Apr 15, 2022
1 parent 6199008 commit 1ebf2f0
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 129 deletions.
10 changes: 0 additions & 10 deletions src/hotspot/share/jvmci/jvmciCompilerToVM.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -898,10 +898,6 @@ C2V_VMENTRY_0(jint, installCode, (JNIEnv *env, jobject, jobject target, jobject
return result;
C2V_END

C2V_VMENTRY_0(jint, getMetadata, (JNIEnv *env, jobject, jobject target, jobject compiled_code, jobject metadata))
JVMCI_THROW_MSG_0(InternalError, "unimplemented");
C2V_END

C2V_VMENTRY(void, resetCompilationStatistics, (JNIEnv* env, jobject))
JVMCICompiler* compiler = JVMCICompiler::instance(true, CHECK);
CompilerStatistics* stats = compiler->stats();
Expand Down Expand Up @@ -1604,10 +1600,6 @@ C2V_VMENTRY_0(jint, methodDataProfileDataSize, (JNIEnv* env, jobject, jlong meta
JVMCI_THROW_MSG_0(IllegalArgumentException, err_msg("Invalid profile data position %d", position));
C2V_END

C2V_VMENTRY_0(jlong, getFingerprint, (JNIEnv* env, jobject, jlong metaspace_klass))
JVMCI_THROW_MSG_0(InternalError, "unimplemented");
C2V_END

C2V_VMENTRY_NULL(jobject, getInterfaces, (JNIEnv* env, jobject, jobject jvmci_type))
if (jvmci_type == NULL) {
JVMCI_THROW_0(NullPointerException);
Expand Down Expand Up @@ -2744,7 +2736,6 @@ JNINativeMethod CompilerToVM::methods[] = {
{CC "getResolvedJavaType0", CC "(Ljava/lang/Object;JZ)" HS_RESOLVED_KLASS, FN_PTR(getResolvedJavaType0)},
{CC "readConfiguration", CC "()[" OBJECT, FN_PTR(readConfiguration)},
{CC "installCode", CC "(" TARGET_DESCRIPTION HS_COMPILED_CODE INSTALLED_CODE "J[B)I", FN_PTR(installCode)},
{CC "getMetadata", CC "(" TARGET_DESCRIPTION HS_COMPILED_CODE HS_METADATA ")I", FN_PTR(getMetadata)},
{CC "resetCompilationStatistics", CC "()V", FN_PTR(resetCompilationStatistics)},
{CC "disassembleCodeBlob", CC "(" INSTALLED_CODE ")" STRING, FN_PTR(disassembleCodeBlob)},
{CC "executeHotSpotNmethod", CC "([" OBJECT HS_NMETHOD ")" OBJECT, FN_PTR(executeHotSpotNmethod)},
Expand All @@ -2766,7 +2757,6 @@ JNINativeMethod CompilerToVM::methods[] = {
{CC "writeDebugOutput", CC "(JIZ)V", FN_PTR(writeDebugOutput)},
{CC "flushDebugOutput", CC "()V", FN_PTR(flushDebugOutput)},
{CC "methodDataProfileDataSize", CC "(JI)I", FN_PTR(methodDataProfileDataSize)},
{CC "getFingerprint", CC "(J)J", FN_PTR(getFingerprint)},
{CC "interpreterFrameSize", CC "(" BYTECODE_FRAME ")I", FN_PTR(interpreterFrameSize)},
{CC "compileToBytecode", CC "(" OBJECTCONSTANT ")V", FN_PTR(compileToBytecode)},
{CC "getFlagValue", CC "(" STRING ")" OBJECT, FN_PTR(getFlagValue)},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -379,22 +379,6 @@ public static CompilerToVM compilerToVM() {
*/
native int installCode(TargetDescription target, HotSpotCompiledCode compiledCode, InstalledCode code, long failedSpeculationsAddress, byte[] speculations);

/**
* Generates the VM metadata for some compiled code and copies them into {@code metaData}. This
* method does not install anything into the code cache.
*
* @param target the target where this code would be installed
* @param compiledCode the result of a compilation
* @param metaData the metadata is written to this object
* @return the outcome of the installation which will be one of
* {@link HotSpotVMConfig#codeInstallResultOk},
* {@link HotSpotVMConfig#codeInstallResultCacheFull},
* {@link HotSpotVMConfig#codeInstallResultCodeTooLarge} or
* {@link HotSpotVMConfig#codeInstallResultDependenciesFailed}.
* @throws JVMCIError if there is something wrong with the compiled code or the metadata
*/
native int getMetadata(TargetDescription target, HotSpotCompiledCode compiledCode, HotSpotMetaData metaData);

/**
* Resets all compilation statistics.
*/
Expand Down Expand Up @@ -688,14 +672,6 @@ HotSpotResolvedObjectTypeImpl getResolvedJavaType(long displacement, boolean com
*/
native int methodDataProfileDataSize(long metaspaceMethodData, int position);

/**
* Gets the fingerprint for a given Klass*.
*
* @param metaspaceKlass
* @return the value of the fingerprint (zero for arrays and synthetic classes).
*/
native long getFingerprint(long metaspaceKlass);

/**
* Return the amount of native stack required for the interpreter frames represented by
* {@code frame}. This is used when emitting the stack banging code to ensure that there is
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -107,12 +107,4 @@ default JavaKind getJavaKind() {

@Override
ResolvedJavaMethod getClassInitializer();

/**
* Gets the fingerprint for this type.
*
* @return the value of the fingerprint ({@code 0} for arrays and synthetic classes or if the VM
* does not support fingerprints)
*/
long getFingerprint();
}
Original file line number Diff line number Diff line change
Expand Up @@ -525,11 +525,6 @@ public int layoutHelper() {
return UNSAFE.getInt(getMetaspaceKlass() + config.klassLayoutHelperOffset);
}

@Override
public long getFingerprint() {
return compilerToVM().getFingerprint(getMetaspaceKlass());
}

synchronized HotSpotResolvedJavaMethod createMethod(long metaspaceHandle) {
long metaspaceMethod = UNSAFE.getLong(metaspaceHandle);
// Maintain cache as array.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -175,11 +175,6 @@ public static int installCode(TargetDescription target,
return CTVM.installCode(target, compiledCode, code, failedSpeculationsAddress, speculations);
}

public static int getMetadata(TargetDescription target,
HotSpotCompiledCode compiledCode, HotSpotMetaData metaData) {
return CTVM.getMetadata(target, compiledCode, metaData);
}

public static void resetCompilationStatistics() {
CTVM.resetCompilationStatistics();
}
Expand Down

1 comment on commit 1ebf2f0

@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.