Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8141552: [TESTBUG] compiler/jvmci/events/JvmciNotifyInstallEventTest failed after jvmci refresh
authordpochepk
Fri, 06 Nov 2015 15:07:00 +0300
changeset 34156 5646275e8053
parent 34155 8d9e0cbf93a2
child 34157 4fde32e81092
child 34158 1f8d643b02d5
8141552: [TESTBUG] compiler/jvmci/events/JvmciNotifyInstallEventTest failed after jvmci refresh Summary: Fixed test bug, which used same CompilationResult object instance in 2 installCode calls Reviewed-by: iignatyev, twisti
hotspot/test/compiler/jvmci/events/JvmciNotifyInstallEventTest.java file | annotate | diff | comparison | revisions 
--- a/hotspot/test/compiler/jvmci/events/JvmciNotifyInstallEventTest.java	Fri Nov 06 14:54:02 2015 +0300
+++ b/hotspot/test/compiler/jvmci/events/JvmciNotifyInstallEventTest.java	Fri Nov 06 15:07:00 2015 +0300
@@ -111,6 +111,8 @@
         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,
jdk-sandbox