RFR(XS): 8155738: C2: fix frame_complete_offset
Lindenmaier, Goetz
goetz.lindenmaier at sap.com
Mon May 2 08:35:26 UTC 2016
Thanks Vladimir!
Best regards,
Goetz.
> -----Original Message-----
> From: Vladimir Kozlov [mailto:vladimir.kozlov at oracle.com]
> Sent: Freitag, 29. April 2016 18:56
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; hotspot compiler
> <hotspot-compiler-dev at openjdk.java.net>
> Subject: Re: RFR(XS): 8155738: C2: fix frame_complete_offset
>
> "In the debug build the call at output.cpp:1831 build if (n->size(_regalloc) <
> (current_offset-instr_offset)) overwrites
> the proper value."
>
> So the fix is to not reset frame_complete_offset
> (Compile::_code_offsets._values[Frame_Complete]) when node is emitted
> into scratch buffer.
>
> Changes looks good.
>
> Thanks,
> Vladimir
>
> On 4/29/16 7:09 AM, Lindenmaier, Goetz wrote:
> > Hi,
> >
> >
> >
> > In C2, frame_complete_offset is set wrong. It's also called during
> scratch_emit_size,
> >
> > which happens at least in the debug build.
> >
> > I also added the missing call on ppc.
> >
> >
> >
> > Please review this small fix. I please need a sponsor:
> >
> > http://cr.openjdk.java.net/~goetz/wr16/8155738-frameCmplt/webrev.00/
> >
> >
> >
> > Thanks and best regards,
> >
> > Goetz.
> >
More information about the hotspot-compiler-dev
mailing list