Code Review for jdk_ea

Prepared by:rrich on Wed Mar 25 22:22:24 CET 2020
Compare against: http://hg.openjdk.java.net/jdk/jdk/
Compare against version:qbase
Summary of changes: 381 lines changed: 200 ins; 89 del; 92 mod; 50049 unchg
Patch of changes: jdk_ea.patch
Bug id: JDK-8227745 : Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/c1/c1_IR.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 362 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/ci/ciEnv.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 475 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/code/compiledMethod.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 748 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/code/debugInfoRec.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 444 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/code/debugInfoRec.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 212 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/code/pcDesc.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 107 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/code/scopeDesc.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 261 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/code/scopeDesc.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 142 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/compiler/compileBroker.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 433 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/jvmci/jvmciCodeInstaller.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 1359 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/opto/c2compiler.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 656 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/opto/callnode.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 1144 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/opto/escape.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 3645 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/opto/machnode.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 1122 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/opto/macro.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 2792 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/opto/matcher.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 2856 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/prims/jni.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
5 lines changed: 0 ins; 5 del; 0 mod; 4216 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/prims/jvm.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
5 lines changed: 0 ins; 5 del; 0 mod; 3699 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/prims/jvmtiEnv.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
5 lines changed: 0 ins; 5 del; 0 mod; 3641 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/prims/jvmtiEnvBase.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 1608 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/prims/jvmtiImpl.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 506 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/deoptimization.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
67 lines changed: 18 ins; 26 del; 23 mod; 3010 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/deoptimization.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
5 lines changed: 0 ins; 1 del; 4 mod; 559 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/interfaceSupport.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 291 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/interfaceSupport.inline.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 516 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/mutex.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
2 lines changed: 0 ins; 1 del; 1 mod; 501 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/mutexLocker.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 376 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/stackValue.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 120 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/thread.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
3 lines changed: 3 ins; 0 del; 0 mod; 5126 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/thread.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 1 ins; 0 del; 0 mod; 2399 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/thread.inline.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 263 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/vframe.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 743 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/vframe_hp.cpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 475 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/vframe_hp.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 171 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/vmOperations.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 1 del; 0 mod; 457 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/utilities/macros.hpp

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 630 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/hotspot/jtreg/serviceability/jvmti/Heap/IterateHeapWithEscapeAnalysisEnabled.java

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
13 lines changed: 0 ins; 11 del; 2 mod; 589 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/hotspot/jtreg/serviceability/jvmti/Heap/libIterateHeapWithEscapeAnalysisEnabled.c

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
1 line changed: 0 ins; 0 del; 1 mod; 379 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/TEST.ROOT

rev 58543 : Changes based on feedback for webrev.3
1 line changed: 1 ins; 0 del; 0 mod; 71 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/com/sun/jdi/EATests.java

rev 58542 : 8227745: Enable Escape Analysis for Better Performance in the Presence of JVMTI Agents
Reviewed-by: ???
rev 58543 : Changes based on feedback for webrev.3
192 lines changed: 124 ins; 34 del; 34 mod; 2945 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/com/sun/jdi/EATestsJVMCI.java

rev 58543 : Changes based on feedback for webrev.3
53 lines changed: 53 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using webrev (vers 25.18-hg+openjdk.java.net).