RFR(S) 8031994: java/lang/Character/CheckProp test times out
Rickard Bäckman
rickard.backman at oracle.com
Tue Jun 3 11:21:44 UTC 2014
Roland, Igor. Thanks for your reviews.
/R
On 06/01, Igor Veresov wrote:
> Looks good. I think it makes sense to emulate the interpreter here since C2 uses the fact the the class was unresolved as one of the profiling points (if it’s unresolved - it’s never been executed).
>
> igor
>
> On Jun 1, 2014, at 8:30 PM, Rickard Bäckman <rickard.backman at oracle.com> wrote:
>
> > Hi,
> >
> > can I please have a couple of review for this change?
> > It starts out as a test that times out, the cause being that ciTypeFlow
> > decides that one of the byte codes will always trap (a new
> > java/io/FileReader) because the class has not been resolved in the
> > constant pool and the compilation we are doing is an OSR compilation
> > with entry far later than the trapping bytecode. This causes the OSR
> > compilation to fail when trying to match blocks.
> >
> > The change makes C1 check if we need to update the constant pool for
> > classes (running with TieredCompilation).
> >
> > Webrev: http://cr.openjdk.java.net/~rbackman/8031994/
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8031994
> >
> > /R
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140603/e9aa060b/signature.asc>
More information about the hotspot-compiler-dev
mailing list