8290065: [JVMCI] only check HotSpotCompiledCode stream is empty if installation succeeds

Doug Simon · Doug Simon · commit 21db9a507b44 · 2022-07-11T16:47:05.000Z
Reviewed-by: kvn
diff --git a/src/hotspot/share/jvmci/jvmciCodeInstaller.cpp b/src/hotspot/share/jvmci/jvmciCodeInstaller.cpp
@@ -692,7 +692,7 @@ JVMCI::CodeInstallResult CodeInstaller::install(JVMCICompiler* compiler,
   JVMCI::CodeInstallResult result = initialize_buffer(compiled_code, buffer, stream, code_flags, JVMCI_CHECK_OK);
 
   u4 available = stream->available();
-  if (available != 0) {
+  if (result == JVMCI::ok && available != 0) {
     JVMCI_ERROR_OK("%d bytes remaining in stream%s", available, stream->context());
   }
 

Original file line number	Diff line number	Diff line change
`@@ -692,7 +692,7 @@ JVMCI::CodeInstallResult CodeInstaller::install(JVMCICompiler* compiler,`
`692`	`692`	`JVMCI::CodeInstallResult result = initialize_buffer(compiled_code, buffer, stream, code_flags, JVMCI_CHECK_OK);`
`693`	`693`
`694`	`694`	`u4 available = stream->available();`
`695`		`- if (available != 0) {`
	`695`	`+ if (result == JVMCI::ok && available != 0) {`
`696`	`696`	`JVMCI_ERROR_OK("%d bytes remaining in stream%s", available, stream->context());`
`697`	`697`	`}`
`698`	`698`