Request for reviews (M): 7083184: JSR 292: don't store context class argument with call site dependencies

John Rose john.r.rose at oracle.com
Thu Aug 25 11:15:03 PDT 2011


Looks good.  -- John

On Aug 25, 2011, at 2:17 AM, Christian Thalinger wrote:

> http://cr.openjdk.java.net/~twisti/7083184/
> 
> 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.
> 
> src/share/vm/ci/ciEnv.cpp
> src/share/vm/ci/ciEnv.hpp
> src/share/vm/code/dependencies.cpp
> src/share/vm/code/dependencies.hpp
> src/share/vm/memory/universe.cpp
> src/share/vm/opto/callGenerator.cpp
> 



More information about the hotspot-compiler-dev mailing list