RFR: 8345269: Fix -Wzero-as-null-pointer-constant warnings in ppc code [v3]

Matthias Baesken mbaesken at openjdk.org
Wed Dec 11 17:52:15 UTC 2024


On Tue, 10 Dec 2024 16:21:22 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Please review this change to remove -Wzero-as-null-pointer-constant warnings
>> in ppc code.
>> 
>> Most places involve just changing literal 0 to nullptr.
>> 
>> Removed a dead return after ShouldNotReachHere() that is no longer needed.
>> 
>> Removed some unused Address constructors that had a default address argument
>> with literal 0 as the default value.  These could have been changed to use
>> nullptr as the default value, but since they aren't used...
>> 
>> Testing:
>> GHA sanity test build, with and without -Wzero-as-null-pointer-constant enabled.
>> I don't have the capability to run tests for this platform, so hoping someone
>> from the aix-ppc maintainers can do more testing.
>
> Kim Barrett 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:
> 
>  - Merge branch 'master' into fix-ppc-zero-null
>  - Merge branch 'master' into fix-ppc-zero-null
>  - Merge branch 'master' into fix-ppc-zero-null
>  - simple ppc fixes
>  - remove unneeded Address ctors

Marked as reviewed by mbaesken (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/22466#pullrequestreview-2496422912


More information about the hotspot-dev mailing list