Integrated: 8255040: Replace __sync_add_and_fetch with __atomic_add_fetch to avoid build errors with clang

Jie Fu jiefu at openjdk.java.net
Tue Oct 27 05:57:20 UTC 2020


On Thu, 22 Oct 2020 11:45:18 GMT, Jie Fu <jiefu at openjdk.org> wrote:

> Hi all,
> 
> It's time to replace __sync_add_and_fetch with __atomic_add_fetch to avoid build errors. 
> 
> After JDK-8252221, build errors with clang were observed [1] due to the use of __sync_add_and_fetch, which is legacy and will be deprecated in the future.
> It can be reproduced by building macos-x86-zero or linux-x86-zero with clang.
> 
> The fix was prepared by learning from aarch64's implementation [2].
> Please review it.
> 
> Thanks
> Best regards,
> Jie
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8255040
> [2] https://github.com/openjdk/jdk/blob/master/src/hotspot/os_cpu/linux_aarch64/atomic_linux_aarch64.hpp#L39

This pull request has now been integrated.

Changeset: d735f919
Author:    Jie Fu <jiefu at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/d735f919
Stats:     13 lines in 2 files changed: 9 ins; 0 del; 4 mod

8255040: Replace __sync_add_and_fetch with __atomic_add_fetch to avoid build errors with clang

Reviewed-by: dholmes, kbarrett

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

PR: https://git.openjdk.java.net/jdk/pull/803


More information about the hotspot-runtime-dev mailing list