[jdk8u-dev] RFR: 8261235: C1 compilation fails with assert(res->vreg_number() == index) failed: conversion check [v2]

[Severin Gehwolf]

On Wed, 12 Mar 2025 08:00:20 GMT, Chuanqi Zang <duke at openjdk.org> wrote:

>> Hi all,
>> 
>> This is backport [JDK-8261235](https://bugs.openjdk.org/browse/JDK-8261235) from jdk17 to jdk8u-dev, to fix C1 compilation failed in HugeMethods.
>> 
>> File 'hotspot/test/compiler/c1/TestTooManyVirtualRegistersMain.java'  remove  annotation '  @requires vm.compiler1.enabled
>> '
>> other file mainly in copyrighht year different.  
>> 
>> it's necessary for jdk8u-dev.
>> 
>> Change has been verified locally, we do test jdk-tier1 tier2 and hotspot-tier1 passed, no risk.
>> 
>>  [JDK-8261235](https://bugs.openjdk.org/browse/JDK-8261235) : C1 compilation fails with assert(res->vreg_number() == index) failed: conversion check
>
> Chuanqi Zang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix codestyle

Why is this an important fix to have in OpenJDK 8u? Why does the risk of adding it at this stage of the lifecycle of 8u not outweigh the benefits?

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

PR Comment: https://git.openjdk.org/jdk8u-dev/pull/622#issuecomment-2886601897