Code Review for 7057587

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).