RFR: 8353192: C2: Clean up x86 backend after 32-bit x86 removal [v2]

Tue Apr 8 17:23:27 UTC 2025

On Tue, 8 Apr 2025 17:11:47 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Aleksey Shipilev 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 five additional commits since the last revision:
>> 
>>  - Cleanup ADLC as well
>>  - Revert some accidental removals
>>  - Merge branch 'master' into JDK-8353192-x86-c2-backend
>>  - Touchup
>>  - Fix
>
> src/hotspot/cpu/x86/x86.ad line 1659:
> 
>> 1657:         return false;
>> 1658:       }
>> 1659:       break;
> 
> So just `!VM_Version::supports_bmi2()` is not possible any more?

It is possible. I just coalesced the cases for `Op_CompressBits` and `Op_ExpandBits`, see a few lines below.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24300#discussion_r2033695690