On Mon, 30 Mar 2026 05:49:32 GMT, Tobias Hartmann <[email protected]> wrote:

>> Benoît Maillard has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Reapply "8380395: [lworld] ProblemList 4 sub-tests of 
>> compiler/valhalla/inlinetypes/TestCallingConvention.java"
>>   
>>   This reverts commit 3a54a623d6c60209a3110fec0a88f32fe56d8d7c.
>
> But the problem list bug id should be updated to 
> [JDK-8380927](https://bugs.openjdk.org/browse/JDK-8380927), right? Also, can 
> we somehow avoid the problem listing by adjusting the test? For example, by 
> setting `-XX:VerifyIterativeGVN=0`. It's unfortunate, that full 
> `TestCallingConvention.java` is problem listed due to this intermittent 
> issue. It will significantly reduce our test coverage.

As discussed offline with @TobiHartmann , we should remove 
`TestCallingConvention.java` from the problemlist to avoid losing too much 
coverage. Because [JDK-8380927](https://bugs.openjdk.org/browse/JDK-8380927) is 
not fixed yet though, I just commented out the test that is failing. I also 
merged `lworld` into this branch, and restarted testing because the previous 
run was kind of a mess.

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

PR Comment: https://git.openjdk.org/valhalla/pull/2242#issuecomment-4169586029

Reply via email to