Request for reviews (XL): 6919934: JSR 292 needs to support x86 C1

[Christian Thalinger]

On Wed, 2010-03-24 at 13:57 -0700, Christian Wimmer wrote:
> Hi Christian, Tom,
> 
> This changeset modifies the GraphBuilder such that now every method  
> has a synthetic exception handler that covers the whole method.  
> Previously, it was only necessary for synchronized methods (and with  
> dtrace). I added the relevant diff below - note that the "if" is  
> remoed by the patch.
> I think this affects the speed of C1 negatively, and perhaps also the  
> generated code (it might slow down the exception dispatch). Why is  
> this change necessary?

Tom would know better as he did the changes, but we changed the code so
that C1 has the same unwinding mechanism as C2 does.  This makes the C1
code a bit simpler.

Regarding performance, why do you think it affects performance
negatively?

-- Christian