Integrated: 8319818: Address GCC 13.2.0 warnings (stringop-overflow and dangling-pointer)

[Mikael Vidstedt]

On Thu, 9 Nov 2023 16:07:53 GMT, Mikael Vidstedt <mikael at openjdk.org> wrote:

> This PR is splitting out the GCC 13.2.0 warning related changes from #16550, excluding the Oracle/devkit parts, for clarity and to make potential backports more straightforward.
> 
> GCC 13.2.0 generates two new warnings:
> 
> * linux-aarch64-debug
> 
> src/hotspot/os_cpu/linux_aarch64/atomic_linux_aarch64.hpp:203:66: error: 'long unsigned int __atomic_load_8(const volatile void*, int)' writing 8 bytes into a region of size 0 overflows the destination [-Werror=stringop-overflow=]
> 
> I did not find any way to adjust the code to avoid this warning, so I instead chose to disable it in `CompileJvm.gmk` for linux-aarch64-(fast)debug only.
> 
> * linux-zero
> 
> src/hotspot/share/runtime/thread.hpp:579:77: error: storing the address of local variable 'rm' in '*_thr_current.Thread::_current_resource_mark' [-Werror=dangling-pointer=]
> 
> Lots and lots of warnings related to ResourceMark. Thanks to @stefank for suggesting moving the ASSERT implementation of the ResourceMark constructor to the .cpp file. With that change there's no need to explicitly disable the warning.

This pull request has now been integrated.

Changeset: c0507af5
Author:    Mikael Vidstedt <mikael at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/c0507af5a4d867940b3aee1ac0fc8188b5536825
Stats:     32 lines in 3 files changed: 19 ins; 10 del; 3 mod

8319818: Address GCC 13.2.0 warnings (stringop-overflow and dangling-pointer)

Reviewed-by: ihse, dholmes

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

PR: https://git.openjdk.org/jdk/pull/16584