RFR: 8292761: x86: Clone nodes to match complex rules [v2]

Quan Anh Mai duke at openjdk.org
Fri Sep 9 11:18:55 UTC 2022


On Fri, 9 Sep 2022 10:09:17 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains ten additional commits since the last revision:
>> 
>>  - Merge branch 'master' into cloneSimpleNodes
>>  - fix
>>  - Merge branch 'master' into cloneSimpleNodes
>>  - shorten
>>  - improve checks
>>  - lea patterns
>>  - refactor
>>  - lea patterns
>>  - first commit
>
> Please include the benchmark in the patch. Could you show the generated code before/after? Thanks!

@TobiHartmann Thanks a lot for your review, I have added the benchmark to the patch, the generated code is as follow:

    CloneNodes::testAndn:
    Baseline:
    movl    $0xffffffff, %r8d
    xorl    0x20(%rbx,%r11,4), %r8d
    movl    %r8d, %r9d
    andl    %edi, %r9d
    andl    %ecx, %r8d

    Patched:
    movl    0x10(%rdi,%r11,4), %r10d
    andnl   %ebx, %r10d, %r9d
    andnl   %ecx, %r10d, %r8d


    CloneNodes::testLea:
    Baseline:
    movl    0x14(%rdi,%r11,4), %r8d
    shll    $0x2, %r8d
    movl    %r8d, %r9d
    addl    $0x7, %r9d
    addl    $0x3, %r8d

    Patched:
    movl    0x10(%rcx,%r11,4), %r10d
    leal    0x3(,%r10,4), %r9d
    leal    0x7(,%r10,4), %r10d

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

PR: https://git.openjdk.org/jdk/pull/9977


More information about the hotspot-compiler-dev mailing list