Request for reviews (M): 7079673: JSR 292: C1 should inline bytecoded method handle adapters

[Tom Rodriguez]

c1_GraphBuilder.cpp:

+   } else if (receiver->as_CheckCast()) {

I think this should be more robust.  The as_Phi and operand_count checks should be part of this guard instead of being asserts.

I assume this will be updated to do the optimization for VCS as well?  Otherwise it looks good.

tom

On Aug 30, 2011, at 9:21 AM, Christian Thalinger wrote:

> http://cr.openjdk.java.net/~twisti/7079673/
> 
> 7079673: JSR 292: C1 should inline bytecoded method handle adapters
> Reviewed-by:
> 
> The current JSR 292 support in C1 always does an invoke for method
> handle invokes which results in a lot of C2I-I2C transfers.  This
> results in very poor performance.
> 
> src/share/vm/c1/c1_GraphBuilder.cpp
> src/share/vm/c1/c1_GraphBuilder.hpp
> src/share/vm/c1/c1_Instruction.cpp
> src/share/vm/c1/c1_Instruction.hpp
> src/share/vm/classfile/javaClasses.cpp
> src/share/vm/classfile/vmSymbols.hpp
>