[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-compiler-dev
Subject:    Re: RFR: 8275086: compiler/c2/irTests/TestPostParseCallDevirtualization.java fails when
From:       Igor Veresov <iveresov () openjdk ! java ! net>
Date:       2021-10-29 18:14:13
Message-ID: jDQ8UE780xJFjGCZwlDX2E73O30df4f7AQM4ydwNB9Q=.6b23c3cd-17a9-4d56-8e0a-0f399c4e0a33 () github ! com
[Download RAW message or body]

On Fri, 29 Oct 2021 09:41:37 GMT, SUN Guoyun <duke@openjdk.java.net> wrote:

> > Hi all,
> > Jtreg test case compiler/c2/irTests/TestPostParseCallDevirtualization.java fails \
> > for fastdebug mode on x86/aarch64/mips architecture when \
> > "--with-jvm-features=-compiler1" be used. the failed info is: 
> > <pre><code class="shell">
> > One or more @IR rules failed:
> > 
> > Failed IR Rules (1)
> > ------------------
> > - Method "public int \
> > compiler.c2.irTests.TestPostParseCallDevirtualization.testMethodHandleCallWithCCP() \
> >                 throws java.lang.Throwable":
> > * @IR rule 1: "@compiler.lib.ir_framework.IR(failOn={"#PRE#(\\\\d+(\\\\s){2}(CallStaticJava.*)+(\\\\s){2}===.*#IS_REPLACED#)", \
> > "invokeBasic"}, applyIf={}, applyIfAnd={}, applyIfOr={}, \
> > counts={"#PRE#(\\\\d+(\\\\s){2}(CallStaticJava.*)+(\\\\s){2}===.*#IS_REPLACED#)", \
> >                 "invokeStatic", "= 1"}, applyIfNot={})" 
> > - failOn: Graph contains forbidden nodes:
> > Regex 1: (\\d+(\\s){2}(CallStaticJava.*)+(\\s){2}===.*invokeBasic)
> > Matched forbidden node:
> > 280  CallStaticJava  ===  5  6  7  8  1 ( 188  1  1  1  1  1  1 ) [[ 281  282  \
> >                 283  285 ]] # Static  java.lang.invoke.MethodHandle::invokeBasic
> > - counts: Graph contains wrong number of nodes:
> > Regex 1: (\\d+(\\s){2}(CallStaticJava.*)+(\\s){2}===.*invokeStatic)
> > Expected 1 but found 0 nodes.
> > 
> > > > > Check stdout for compilation output of the failed methods
> > </code></pre>
> > 
> > This is a patch to fix this problem. Please help review it.
> > 
> > Thanks,
> > Sun Guoyun
> 
> SUN Guoyun has updated the pull request incrementally with one additional commit \
> since the last revision: 
> 8275086: compiler/c2/irTests/TestPostParseCallDevirtualization.java fails when \
> compiler1 is disabled

So, now 5000 warmup iterations help? You mentioned previously that it had to be 20000 \
? What changed?

-------------

PR: https://git.openjdk.java.net/jdk/pull/5903


[prev in list] [next in list] [prev in thread] [next in thread]