Prepared by: | never on Wed Jun 22 10:38:58 PDT 2011 |
---|---|
Workspace: | /net/smite.us.oracle.com/export/ws/7045514 |
Compare against: | ssh://hg.openjdk.java.net/hsx/hotspot-comp-gate/hotspot |
Summary of changes: | 40 lines changed: 6 ins; 0 del; 34 mod; 4861 unchg |
Patch of changes: | 7057587.patch |
Author comments: |
7057587: JSR 292 - crash with jruby in test/test_respond_to.rb Summary: don't skip receiver when GC'ing compiled invokedynamic callsites Reviewed-by: When GC'ing at a call site during resolution the arguments to the call have to be handled specially. In most cases the implicit receiver to method handle invoke is ignored but in this case it must be treated as real so that it is properly GC'ed. Tested with failing test from report. Also ran jck, regression tests and vm.mlvm tests. I also included a fix to the inline level printing. For method handle invokes we don't want to count them against MaxInlineLevel and previously this was done by adding a bias to the inline_depth. This screwed up the indenting making the inlining output unreadable. Instead we should keep the depth count consistent and adjust the max inline level for the subtree. This is done by keeping that value in the InlineTree. inline_depth was renamed to inline_level to be consistent with MaxInlineLevel. |
Bug id: | Bug Database |
Legend: |
Modified file Deleted file New file |
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/code/nmethod.cpp
2 lines changed: 1 ins; 0 del; 1 mod; 2780 unchg
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/opto/parse.hpp
4 lines changed: 0 ins; 0 del; 4 mod; 586 unchg
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/opto/bytecodeInfo.cpp
33 lines changed: 5 ins; 0 del; 28 mod; 547 unchg
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/share/vm/opto/doCall.cpp
1 line changed: 0 ins; 0 del; 1 mod; 948 unchg
This code review page was prepared using /never/bin/webrev (vers 23.18-hg-never).