Request for reviews (XXL): 6893268: additional dynamic language related optimizations in C2
Vladimir Kozlov
Vladimir.Kozlov at Sun.COM
Tue Dec 1 14:51:04 PST 2009
Christian Thalinger wrote:
> And this one adds C2 inlining support:
>
> http://cr.openjdk.java.net/~twisti/6893268/webrev.01/
>
> -- Christian
>
src/share/vm/ci/ciExceptionHandler.cpp
I think you need to add assert since you removed
assert(accessor->get_instanceKlass()->is_linked()) in ciEnv.cpp
src/share/vm/runtime/frame.hpp
rename is_static to has_receiver in oops_compiled_arguments_do()
src/share/vm/prims/methodHandleWalk.hpp
remove commented lines
! // union {
! jvalue _value;
! Handle _handle;
! // } _value;
Vladimir
More information about the hotspot-compiler-dev
mailing list