review for 7055355: JSR 292: crash while throwing WrongMethodTypeException

John Rose john.r.rose at
Wed Jun 15 23:22:36 PDT 2011

Reviewed.  Thanks for fixing this.   -- John

On Jun 15, 2011, at 10:32 PM, Tom Rodriguez wrote:

> 174 lines changed: 54 ins; 111 del; 9 mod; 30934 unchg
> 7055355: JSR 292: crash while throwing WrongMethodTypeException
> Reviewed-by:
> When throwing a WrongMethodTypeException from a MethodHandle the
> existing code was pretending it was in the interpreter which worked ok
> most of the time but would sometimes die when trying to throw with a
> compiled caller.  It could cause asserts in some contexts or result in
> GC crashes.  The fix is to remove the old machinery in favor of the
> more standard code for throwing exceptions in
> StubRoutines/SharedRuntime.  There was one other use of the old throw
> machinery but it's in part of check that can never fail since we won't
> enable method handles unless we have a non-NULL rasie exception
> method.  There's an extra fix in the there to reject lookups of
> clinit.  It isn't critical but it allows all the JCK tests to run
> without asserting.  Tested with failing JCK test and jruby crasher on
> sparc/x86 with client/server/d64, along with a full run of the
> invokedynamic JCK tests, the JLI regression tests and the vm.mlvm
> tests.

More information about the hotspot-compiler-dev mailing list