[8u-dev] RFA: 8206075: On x86, assert on unbound assembler Labels used as branch targets

Hohensee, Paul hohensee at amazon.com
Wed Dec 12 17:43:25 UTC 2018


I pushed backports of 8206075, 8208480, and 8208647 to jdk11u. Combined request:

Please approve this small backport to jdk8u.

8206075: On x86, assert on unbound assembler Labels used as branch targets
8208480: Test failure: assert(is_bound() || is_unused()) after JDK-8206075 in C1
JBS: https://bugs.openjdk.java.net/browse/JDK-8206075 https://bugs.openjdk.java.net/browse/JDK-8206075
Webrev: http://cr.openjdk.java.net/~phh/8206075.8208480/webrev.00/
Review thread: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2018-July/029136 http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2018-August/029627.html

The combined patch applies cleanly net of line number changes and file locations.

Thanks,
Paul

On 12/5/18, 3:23 PM, "jdk8u-dev on behalf of Hohensee, Paul" <jdk8u-dev-bounces at openjdk.java.net on behalf of hohensee at amazon.com> wrote:

    Yes. I've filed jdk11u requests for both, as well as JDK-8208647, which latter is necessary to build with the latest jtreg that I've got installed on my laptop.
    
    Thanks,
    
    Paul
    
    On 12/3/18, 4:15 AM, "Seán Coffey" <sean.coffey at oracle.com> wrote:
    
        Let's put the 8206075/8208480 request together for porting purposes.
        
        Can this be fixed in the JDK 11 Updates repository first ?
        
        Regards,
        Sean.
        
        On 08/11/18 00:29, Hohensee, Paul wrote:
        > Email is asynchronous and this message lost the race. :)
        >
        > On 11/7/18, 4:25 PM, "jdk8u-dev on behalf of Hohensee, Paul" <jdk8u-dev-bounces at openjdk.java.net on behalf of hohensee at amazon.com> wrote:
        >
        >      
        >      Review thread: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2018-July/029136. Please approve this small backport to jdk8u. The original patch had a bug that was fixed by
        >      8208480: Fix for test failure: assert(is_bound() || is_unused()) after JDK-8206075 in C1
        >      for which a backport request immediately follows this one.
        >      JBS: https://bugs.openjdk.java.net/browse/JDK-8206075
        >      Webrev: http://cr.openjdk.java.net/~phh/8206075/webrev.8u.00/html
        >      
        >      The patch applies cleanly net of line number changes and file locations.
        >      
        >      Webrev combined with 8208480: http://cr.openjdk.java.net/~phh/8206075/webrev.8u.8208480.combined/
        >      
        >      Thanks,
        >      
        >      Paul
        >      
        >      
        >
        
        
    
    



More information about the jdk8u-dev mailing list