On Sun, 13 Nov 2022 23:16:47 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> After [JDK-8292008](https://bugs.openjdk.org/browse/JDK-8292008) and 
>> [JDK-8247283](https://bugs.openjdk.org/browse/JDK-8247283), some C and C++ 
>> code across the JDK can be replaced and simplified with cleaner language 
>> features that were previously not available due to required compatibility 
>> with the now unsupported Visual C++ 2017 compiler. These cleanups were 
>> highlighted by the very briefly integrated 8296115
>> 
>> No changes to the behaviour of the JDK has resulted in any way from this 
>> commit
>
> make/autoconf/flags-cflags.m4 line 632:
> 
>> 630:   if test "x$TOOLCHAIN_TYPE" = xgcc || test "x$TOOLCHAIN_TYPE" = 
>> xclang; then
>> 631:     STATIC_LIBS_CFLAGS="$STATIC_LIBS_CFLAGS -ffunction-sections 
>> -fdata-sections \
>> 632:       -DJNIEXPORT='[[gnu::visibility(\"hidden\")]]'"
> 
> So IIUC we now use attributes via the C++11 syntax rather than 
> compiler-specific syntax - even where the C++11 syntax is referring to a 
> compiler specific attribute. Is that right?

Yep, just something that C++ does a little neater, at least in my view

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

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

Reply via email to