RFR: 8375241: Simplify --with-native-debug-symbols-level option implementation [v2]
Aleksey Shipilev
shade at openjdk.org
Mon Jan 19 07:54:26 UTC 2026
On Mon, 19 Jan 2026 07:04:36 GMT, Julian Waters <jwaters at openjdk.org> wrote:
>> --with-native-debug-symbols-level was added some time ago to allow selecting the level of debugging information created for compiled binaries. The UTIL_ARG_WITH utility that the option implementation uses has several features that can be used to simplify handling of this option, so that is done in this change.
>
> Julian Waters has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove -g and don't pass the flag at all if option is not given flags-cflags.m4
make/autoconf/flags-cflags.m4 line 85:
> 83: ],
> 84: IF_NOT_GIVEN: [
> 85: NATIVE_DEBUG_SYMBOLS_LEVEL=""
I would have thought this needs to be `-g`, so we pass `-g` when `--native-debug-symbols-level` is not specified?
make/autoconf/flags-cflags.m4 line 113:
> 111:
> 112: # Debug info level should follow the debug format to be effective.
> 113: CFLAGS_DEBUG_SYMBOLS="-gdwarf-4 ${NATIVE_DEBUG_SYMBOLS_LEVEL}"
Yeah, from this definition, it is not a "level", it is "flags". So name it e.g. `NATIVE_DEBUG_SYMBOLS_FLAGS` or some such?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29214#discussion_r2703626427
PR Review Comment: https://git.openjdk.org/jdk/pull/29214#discussion_r2703629603
More information about the build-dev
mailing list