Code Review for 7083184

Prepared by:twisti on Thu Aug 25 01:58:59 PDT 2011
Workspace:/home/twisti/hsx/hotspot-comp/7083184
Compare against: /home/twisti/hsx/hotspot-comp/hotspot
Summary of changes: 145 lines changed: 64 ins; 12 del; 69 mod; 6476 unchg
Patch of changes: 7083184.patch
Printable review: 7083184.pdf
Author comments:
7083184: JSR 292: don't store context class argument with call site dependencies
Reviewed-by:

The changes of 7071653 store a context class argument per call site
dependency in the dependency stream. This is actually not required
since the context class is implicitly available with the first
argument; the call site object. Additionally call site dependencies
should not depend on the very general super class CallSite but rather
its actual class.

Bug id: Bug Database
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw src/share/vm/ci/ciEnv.cpp

34 lines changed: 15 ins; 4 del; 15 mod; 1143 unchg

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

3 lines changed: 0 ins; 0 del; 3 mod; 406 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw src/share/vm/code/dependencies.cpp

74 lines changed: 36 ins; 8 del; 30 mod; 1593 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw src/share/vm/code/dependencies.hpp

23 lines changed: 13 ins; 0 del; 10 mod; 607 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw src/share/vm/memory/universe.cpp

3 lines changed: 0 ins; 0 del; 3 mod; 1631 unchg

Cdiffs Udiffs Wdiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/callGenerator.cpp

8 lines changed: 0 ins; 0 del; 8 mod; 1096 unchg

This code review page was prepared using /home/twisti/bin/webrev (vers 23.18-hg-never).