Hi Jiangli, that's really sad!
First because it is not the way it is supposed to work and second because we detected and fixed this same problem a few days ago as well.. I don't understand what's the problem to post a review request on the list and get the reviews by mail. That shouldn't take much longer than a verbal review, even if you and the reviewers all sit in the same room. Regards, Volker On Wed, Apr 24, 2013 at 6:26 PM, Jiangli Zhou <jiangli.z...@oracle.com> wrote: > Hi David, > > It was an urgent fix and didn't go through the "normal" route. > > Thanks, > Jiangli > > > On 04/23/2013 09:32 PM, David Holmes wrote: >> >> Jiangli, >> >> I do not see a public Request for Review for this change. >> >> David >> >> On 24/04/2013 9:11 AM, jiangli.z...@oracle.com wrote: >>> >>> Changeset: 1ea6a35dcbe5 >>> Author: jiangli >>> Date: 2013-04-23 12:32 -0400 >>> URL: http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/1ea6a35dcbe5 >>> >>> 8012927: 'assert(nbits == 32 || (-(1 << nbits-1) <= x && x < ( 1 << >>> nbits-1))) failed: value out of range' in interpreter initialization. >>> Summary: Change br_null_short() to br_null(). >>> Reviewed-by: coleenp, hseigel >>> >>> ! src/cpu/sparc/vm/interp_masm_sparc.cpp >>> >
