review request (L): 6981777 implement JSR 292 EG adjustments from summer 2010

Christian Thalinger christian.thalinger at oracle.com
Mon Sep 20 03:31:02 PDT 2010


On Mon, 2010-09-20 at 02:25 -0700, John Rose wrote:
> On Sep 20, 2010, at 12:57 AM, Christian Thalinger wrote:
> 
> > src/share/vm/classfile/classFileParser.cpp:
> > 
> > +          cfs->guarantee_more(3, CHECK);  // operand_count, ..., tag/access_flags
> > +          u2 op_count = cfs->get_u2_fast();
> > +          cfs->guarantee_more(2*op_count + 1, CHECK);
> > 
> > I think the additional +1 is wrong as you check for 3 in the first
> > guarantee_more.
> 
> Yes, but that 3 only assures the tag will be present if op_count==0.
> The way this big loop works is everybody has to ensure that the next
> guy's tag is already loaded.
> So almost every guarantee_more has to have a +1.

Makes sense.  -- Christian



More information about the hotspot-compiler-dev mailing list