RFR: 8318078: ADLC: pass ASSERT and PRODUCT flags [v2]

Emanuel Peter epeter at openjdk.org
Mon Oct 16 10:34:38 UTC 2023


On Fri, 13 Oct 2023 17:23:32 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   add comments like Vladimir requested
>
> make/hotspot/gensrc/GensrcAdlc.gmk line 138:
> 
>> 136:   # Set ASSERT, NDEBUG and PRODUCT flags just like in JvmFlags.gmk
>> 137:   ifeq ($(DEBUG_LEVEL), release)
>> 138:     ADLCFLAGS += -DNDEBUG
> 
> May be you should also copy all comments from `JvmFlags.gmk` to avoid confusion because, for example, `NDEBUG` is not used directly in HotSpot code but it is needed "to disable uses of assert macro from <assert.h>."

Done

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16178#discussion_r1360341172


More information about the build-dev mailing list