Request for reviews (M): 7083184: JSR 292: don't store context class argument with call site dependencies
Tom Rodriguez
tom.rodriguez at oracle.com
Thu Aug 25 13:01:58 PDT 2011
Looks good.
tom
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