Integrated: 8345757: [ASAN] clang17 report 'dprintf' macro redefined

SendaoYan syan at openjdk.org
Wed Dec 25 01:32:48 UTC 2024


On Sun, 8 Dec 2024 11:50:13 GMT, SendaoYan <syan at openjdk.org> wrote:

> Hi all,
> File src/java.smartcardio/share/native/libj2pcsc/pcsc.c and src/jdk.crypto.cryptoki/share/native/libj2pkcs11/j2secmod.h generate compile warning `'dprintf' macro redefined` by clang17/llvm17 with fastdebug configure. This PR rename dprintf to debug_printf and make it variadic, to avoid deplication macro definition to linux header `/usr/include/bits/stdio2.h`, risk is low.
> 
> Additional testing:
> 
> - [x] build with fastdebug and release configure by gcc14 on linux-x64
> - [x] jtreg tests(include tier1/2/3 etc.) on linux-x64 with release build
> - [x] jtreg tests(include tier1/2/3 etc.) on linux-x64 with fastdebug build
> - [x] jtreg tests(include tier1/2/3 etc.) on linux-aarch64 with release build
> - [x] jtreg tests(include tier1/2/3 etc.) on linux-aarch64 with fastdebug build

This pull request has now been integrated.

Changeset: 6c591854
Author:    SendaoYan <syan at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/6c59185475eeca83153f085eba27cc0b3acf9bb4
Stats:     52 lines in 5 files changed: 0 ins; 12 del; 40 mod

8345757: [ASAN] clang17 report 'dprintf' macro redefined

Reviewed-by: kbarrett, valeriep

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

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


More information about the build-dev mailing list