RFR: 8234617: C1: Incorrect result of field load due to missing narrowing conversion

[Hohensee, Paul]

Lgtm, assuming TestPrimitiveConversions.java passes. Worth doing because it's a simple fix, for a bug which can manifest as data corruption.

Thanks,
Paul

On 6/8/20, 12:24 PM, "jdk8u-dev on behalf of Sergey Nazarkin" <jdk8u-dev-bounces at openjdk.java.net on behalf of snazarkin at azul.com> wrote:

    Original change
    https://jira.azulsystems.com/browse/JDK-8234617

    I’m not sure 8u need this but the fix reminds me recent BBB issue.

    If we need this the patch is following
    webrev
    http://cr.openjdk.java.net/~snazarki/jdk8u-dev-webrev/8234617/

    c1_GraphBuilder is patched cleanly after path correction
    TestPrimitiveConversions.java needs correct Asserts.java path


    Sergey