Integrated: 8367282: FORBID_C_FUNCTION needs exception spec consistent with library declaration

Kim Barrett kbarrett at openjdk.org
Fri Sep 12 19:00:25 UTC 2025


On Wed, 10 Sep 2025 02:58:31 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this change to FORBID_C_FUNCTION to make the exception specs of
> the forbidding declarations match up with the exception specs of the
> associated library declarations. This needs to account for different platform
> libraries either having or not having exception specs. It's needed because,
> after switching to C++17, some compilers complain about some differences in
> the exception specs.
> 
> Also removed the workaround for JDK-8367051, which should no longer be needed
> after this change.
> 
> Hoping someone will test the aix-ppc port. @MBaesken or @TheRealMDoerr ?
> 
> Testing: mach5 tier1, GHA sanity checks (nearly finished)
> Locally (linux-aarch64) built with clang, which failed without the JDK-8367051
> workaround.

This pull request has now been integrated.

Changeset: 4e59c63e
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/4e59c63ec5a896a09f61a019e2fc5a2ec75ec40e
Stats:     73 lines in 4 files changed: 31 ins; 9 del; 33 mod

8367282: FORBID_C_FUNCTION needs exception spec consistent with library declaration

Reviewed-by: dholmes, mbaesken

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

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


More information about the hotspot-dev mailing list