changeset 10000:ccc3799ab45e

8144529: [JVMCI] compiler/jvmci/errors/TestInvalidCompilationResult.java fails to compile after JDK-8143730 Reviewed-by: kvn
author twisti
date Wed, 02 Dec 2015 14:59:57 -1000
parents 37ee45971b17
children 391e6ba2b8ad
files test/compiler/jvmci/errors/CodeInstallerTest.java test/compiler/jvmci/errors/TestInvalidCompilationResult.java test/compiler/jvmci/events/JvmciNotifyInstallEventTest.java
diffstat 3 files changed, 2 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/test/compiler/jvmci/errors/CodeInstallerTest.java	Wed Dec 02 11:40:18 2015 -1000
+++ b/test/compiler/jvmci/errors/CodeInstallerTest.java	Wed Dec 02 14:59:57 2015 -1000
@@ -68,6 +68,7 @@
     }
 
     protected void installCode(CompilationResult result) {
+        result.close();
         codeCache.addCode(dummyMethod, result, null, null);
     }
 
--- a/test/compiler/jvmci/errors/TestInvalidCompilationResult.java	Wed Dec 02 11:40:18 2015 -1000
+++ b/test/compiler/jvmci/errors/TestInvalidCompilationResult.java	Wed Dec 02 14:59:57 2015 -1000
@@ -219,13 +219,6 @@
     }
 
     @Test(expected = JVMCIError.class)
-    public void testUnknownInfopointReason() {
-        CompilationResult result = createEmptyCompilationResult();
-        result.addInfopoint(new Infopoint(0, null, InfopointReason.UNKNOWN));
-        installCode(result);
-    }
-
-    @Test(expected = JVMCIError.class)
     public void testInfopointMissingDebugInfo() {
         CompilationResult result = createEmptyCompilationResult();
         result.addInfopoint(new Infopoint(0, null, InfopointReason.METHOD_START));
--- a/test/compiler/jvmci/events/JvmciNotifyInstallEventTest.java	Wed Dec 02 11:40:18 2015 -1000
+++ b/test/compiler/jvmci/events/JvmciNotifyInstallEventTest.java	Wed Dec 02 14:59:57 2015 -1000
@@ -106,13 +106,12 @@
         HotSpotCompilationRequest compRequest = new HotSpotCompilationRequest(method, -1, 0L);
         // to pass sanity check of default -1
         compResult.setTotalFrameSize(0);
+        compResult.close();
         codeCache.installCode(compRequest, compResult, /* installedCode = */ null, /* speculationLog = */ null,
                 /* isDefault = */ false);
         Asserts.assertEQ(gotInstallNotification, 1,
                 "Got unexpected event count after 1st install attempt");
         // since "empty" compilation result is ok, a second attempt should be ok
-        compResult = new CompilationResult(METHOD_NAME); // create another instance with fresh state
-        compResult.setTotalFrameSize(0);
         codeCache.installCode(compRequest, compResult, /* installedCode = */ null, /* speculationLog = */ null,
                 /* isDefault = */ false);
         Asserts.assertEQ(gotInstallNotification, 2,