RFR: 8368961: Remove redundant checks in ciField.cpp
Anton Seoane Ampudia
aseoane at openjdk.org
Thu Nov 13 09:36:03 UTC 2025
On Wed, 12 Nov 2025 05:01:03 GMT, Zihao Lin <duke at openjdk.org> wrote:
>> @linzihao1999 I noted there is another more comprehensive issue for this function: https://bugs.openjdk.org/browse/JDK-8368961
>>
>> There are a total of 3 redundant checks in this function that can be removed. If you want to update this patch, feel free to update the issue for the PR, and include the cleanup for all 3 redundant checks.
>
>> @linzihao1999 I noted there is another more comprehensive issue for this function: https://bugs.openjdk.org/browse/JDK-8368961
>>
>> There are a total of 3 redundant checks in this function that can be removed. If you want to update this patch, feel free to update the issue for the PR, and include the cleanup for all 3 redundant checks.
>
> Sure, I will update this change.
@linzihao1999 please note that you will need a "capital R" Reviewer to approve your changes before being able to integrate (you can see the required progress in the body of your PR)
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28191#issuecomment-3526752066
More information about the hotspot-compiler-dev
mailing list