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

List:       openjdk-compiler-dev
Subject:    Re: [8u] RFR Backport of 8067429: java.lang.VerifyError: Inconsistent stackmap frames at branch targ
From:       Andrew Leonard <andrew_m_leonard () uk ! ibm ! com>
Date:       2019-08-20 8:45:34
Message-ID: OF0A8CBD6D.4451AC84-ON0025845C.00301164-8025845C.00301EA2 () notes ! na ! collabserv ! com
[Download RAW message or body]

This is a multipart message in MIME format.

--=_alternative 00301E478025845C_=
Content-Type: text/plain; charset="US-ASCII"

Thank you Vincente,
i'll request approval..
Cheers
Andrew

Andrew Leonard
Java Runtimes Development
IBM Hursley
IBM United Kingdom Ltd
internet email: andrew_m_leonard@uk.ibm.com 




From:   Vicente Romero <vicente.romero@oracle.com>
To:     Andrew Leonard <andrew_m_leonard@uk.ibm.com>, 
jdk8u-dev@openjdk.java.net
Cc:     compiler-dev@openjdk.java.net
Date:   19/08/2019 20:00
Subject:        Re: [8u] RFR Backport of 8067429: java.lang.VerifyError: 
Inconsistent stackmap frames at branch target



the backport looks good to me,

Vicente

On 8/15/19 4:38 AM, Andrew Leonard wrote:
> Any offers to review this please?
> Thanks
> Andrew
> 
> 
> 
> 
> From:   Andrew Leonard/UK/IBM
> To:     jdk8u-dev@openjdk.java.net
> Date:   12/08/2019 17:15
> Subject:        [8u] RFR Backport of 8067429: java.lang.VerifyError:
> Inconsistent stackmap frames at branch target
> 
> 
> Hi,
> Please can I request a review of this updated patch for 8067429 to
> backport to jdk8u/langtools. We have seen problem reports from clients
> seeing this issue with jdk8 and would like to request a backport please.
> The patch is essentially the same as published to jdk9+, but jdk8 the 
file
> paths are in a different location. I have built and tested all "tier1" 
and
> "langtools_all" successfully, and the patch new testcase passes.
> 
https://urldefense.proofpoint.com/v2/url?u=http-3A__cr.openjdk.java.net_-7Ealeonard_80 \
67429_webrev.00&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=NaV8Iy8Ld-vjpXZFDdTbgGlRTghGHnwM75 \
wUPd5_NUQ&m=1jq899yhiTZEKwvfBlQDLl4IJLHPNwrWr9Ee9y-TSek&s=OxsfFEnFwWH4k-vJMXXgjYaUISo-OM0CpI8bn_PRNoY&e= \


> Once reviewed I will add the jdk8u-fix-request.
> 
> Many thanks
> Andrew
> 
> 
> Andrew Leonard
> Java Runtimes Development
> IBM Hursley
> IBM United Kingdom Ltd
> internet email: andrew_m_leonard@uk.ibm.com
> 
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU
> 
> 
> 
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU
> 
> 
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU
> 
> 
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU
> 





Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 
741598. 
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU


--=_alternative 00301E478025845C_=
Content-Type: text/html; charset="US-ASCII"
Content-Transfer-Encoding: 
X-Proofpoint-UnRewURL: 13 URL's were un-rewritten

<span style=" font-size:10pt;font-family:sans-serif">Thank you Vincente,</span>
<br><span style=" font-size:10pt;font-family:sans-serif">i'll request \
approval..</span> <br><span style=" \
font-size:10pt;font-family:sans-serif">Cheers</span> <br><span style=" \
font-size:10pt;font-family:sans-serif">Andrew</span> <br>
<br><span style=" font-size:12pt">Andrew Leonard<br>
Java Runtimes Development<br>
IBM Hursley<br>
IBM United Kingdom Ltd<br>
internet email: andrew_m_leonard@uk.ibm.com </span>
<br>
<br>
<br>
<br>
<br><span style=" font-size:9pt;color:#5f5f5f;font-family:sans-serif">From:
&nbsp; &nbsp; &nbsp; &nbsp;</span><span style=" \
font-size:9pt;font-family:sans-serif">Vicente Romero \
&lt;vicente.romero@oracle.com&gt;</span> <br><span style=" \
font-size:9pt;color:#5f5f5f;font-family:sans-serif">To: &nbsp; &nbsp; &nbsp; \
&nbsp;</span><span style=" font-size:9pt;font-family:sans-serif">Andrew Leonard \
&lt;andrew_m_leonard@uk.ibm.com&gt;, jdk8u-dev@openjdk.java.net</span> <br><span \
style=" font-size:9pt;color:#5f5f5f;font-family:sans-serif">Cc: &nbsp; &nbsp; &nbsp; \
&nbsp;</span><span style=" \
font-size:9pt;font-family:sans-serif">compiler-dev@openjdk.java.net</span> <br><span \
style=" font-size:9pt;color:#5f5f5f;font-family:sans-serif">Date: &nbsp; &nbsp; \
&nbsp; &nbsp;</span><span style=" font-size:9pt;font-family:sans-serif">19/08/2019 \
20:00</span> <br><span style=" \
font-size:9pt;color:#5f5f5f;font-family:sans-serif">Subject: &nbsp; &nbsp; &nbsp; \
&nbsp;</span><span style=" font-size:9pt;font-family:sans-serif">Re: [8u] RFR \
Backport of 8067429: java.lang.VerifyError: Inconsistent stackmap frames at branch \
target</span> <br>
<hr noshade>
<br>
<br>
<br><tt><span style=" font-size:10pt">the backport looks good to me,<br>
<br>
Vicente<br>
<br>
On 8/15/19 4:38 AM, Andrew Leonard wrote:<br>
&gt; Any offers to review this please?<br>
&gt; Thanks<br>
&gt; Andrew<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; From: &nbsp; Andrew Leonard/UK/IBM<br>
&gt; To: &nbsp; &nbsp; jdk8u-dev@openjdk.java.net<br>
&gt; Date: &nbsp; 12/08/2019 17:15<br>
&gt; Subject: &nbsp; &nbsp; &nbsp; &nbsp;[8u] RFR Backport of 8067429:
java.lang.VerifyError:<br>
&gt; Inconsistent stackmap frames at branch target<br>
&gt;<br>
&gt;<br>
&gt; Hi,<br>
&gt; Please can I request a review of this updated patch for 8067429 to<br>
&gt; backport to jdk8u/langtools. We have seen problem reports from clients<br>
&gt; seeing this issue with jdk8 and would like to request a backport please.<br>
&gt; The patch is essentially the same as published to jdk9+, but jdk8
the file<br>
&gt; paths are in a different location. I have built and tested all &quot;tier1&quot;
and<br>
&gt; &quot;langtools_all&quot; successfully, and the patch new testcase
passes.<br>
&gt; </span></tt><a href="http://cr.openjdk.java.net/~aleonard/8067429/webrev.00"><tt><span \
style=" font-size:10pt">http://cr.openjdk.java.net/~aleonard/8067429/webrev.00</span></tt></a><tt><span \
style=" font-size:10pt"> <br>
&gt; Once reviewed I will add the jdk8u-fix-request.<br>
&gt;<br>
&gt; Many thanks<br>
&gt; Andrew<br>
&gt;<br>
&gt;<br>
&gt; Andrew Leonard<br>
&gt; Java Runtimes Development<br>
&gt; IBM Hursley<br>
&gt; IBM United Kingdom Ltd<br>
&gt; internet email: andrew_m_leonard@uk.ibm.com<br>
&gt;<br>
&gt; Unless stated otherwise above:<br>
&gt; IBM United Kingdom Limited - Registered in England and Wales with
number<br>
&gt; 741598.<br>
&gt; Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire
PO6 3AU<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; Unless stated otherwise above:<br>
&gt; IBM United Kingdom Limited - Registered in England and Wales with
number<br>
&gt; 741598.<br>
&gt; Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire
PO6 3AU<br>
&gt;<br>
&gt;<br>
&gt; Unless stated otherwise above:<br>
&gt; IBM United Kingdom Limited - Registered in England and Wales with
number<br>
&gt; 741598.<br>
&gt; Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire
PO6 3AU<br>
&gt;<br>
&gt;<br>
&gt; Unless stated otherwise above:<br>
&gt; IBM United Kingdom Limited - Registered in England and Wales with
number<br>
&gt; 741598.<br>
&gt; Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire
PO6 3AU<br>
&gt;<br>
<br>
<br>
</span></tt>
<br>
<br><span style=" font-size:10pt;font-family:sans-serif"><br>
Unless stated otherwise above:<br>
IBM United Kingdom Limited - Registered in England and Wales with number
741598. <br>
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6
3AU<br>
</span>

--=_alternative 00301E478025845C_=--


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

Configure | About | News | Add a list | Sponsored by KoreLogic