RFR: 8316893: Compile without -fno-delete-null-pointer-checks [v2]

Daniel Jeliński djelinski at openjdk.org
Mon Oct 2 08:26:52 UTC 2023


On Sun, 1 Oct 2023 05:13:25 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix typo
>
> make/autoconf/flags-cflags.m4 line 923:
> 
>> 921: AC_DEFUN([FLAGS_SETUP_GCC6_COMPILER_FLAGS],
>> 922: [
>> 923:   # This flags is required for GCC 6 builds as undefined behavior in OpenJDK code
> 
> s/flags/flag/

Oops, missed it. Thanks for pointing it out!

> make/hotspot/lib/CompileJvm.gmk line 165:
> 
>> 163:     DISABLED_WARNINGS_gcc := $(DISABLED_WARNINGS_gcc), \
>> 164:     DISABLED_WARNINGS_gcc_ad_$(HOTSPOT_TARGET_CPU_ARCH).cpp := nonnull, \
>> 165:     DISABLED_WARNINGS_gcc_ad_$(HOTSPOT_TARGET_CPU_ARCH)_pipeline.cpp := nonnull-compare, \
> 
> How about just waiting for the fix for JDK-8316907 to go in before doing this one, rather than adding these
> temporary warning disables.

Yeah, that's what I wanted to do initially. But then the review on JDK-8316907 failed to make progress, so I figured I'd race this PR with that one.
I will remove the warning suppressions before merging the second PR, whichever that is.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15982#discussion_r1342351752
PR Review Comment: https://git.openjdk.org/jdk/pull/15982#discussion_r1342351782


More information about the build-dev mailing list