Hi All,

I try to send a patch via thunderbird, I don't know if the process is correct.
I tried from gmail but it was not good.

The patch is trivial and I use it to learn the patch submission process, along 
with the signed-off-by process

Cheers,
Phi
From 3ac65cf6eec36a9a22af378e89bc4d5b148d8713 Mon Sep 17 00:00:00 2001
From: Philippe Benard <phi.deb...@gmail.com>
Date: Thu, 9 Jan 2025 14:59:00 +0100
Subject: [PATCH] libcpu: Remove false positive null ptr deref warning for GCC
 -O3

See bugzilla Bug#32527

* libcpu/Makefile.am
i386_lex_CFLAGS += $(subst -O3,-Wno-null-dereference,$(findstring -O3,$(CFLAGS)))

i386_lex_CFLAGS is a make variable to tweak the i386_lex.c.o
specific compile flags. The are already some -W defined there, I'd
like to add one to avoid seeing false warning during build with -O3
compile flag.

GCC 14.2 (as well as 13.x) produce a false positive warning for the
i386 lexer, a bug is filled already against GCC for that, CLANG is
imune.

Signed-off-by: Philippe Benard <phi.deb...@gmail.com>
---
 libcpu/Makefile.am | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libcpu/Makefile.am b/libcpu/Makefile.am
index 32835237..2e87021b 100644
--- a/libcpu/Makefile.am
+++ b/libcpu/Makefile.am
@@ -91,6 +91,8 @@ libeu = ../lib/libeu.a
 
 i386_lex_CFLAGS = -Wno-unused-label -Wno-unused-function -Wno-sign-compare \
 		  -Wno-implicit-fallthrough
+i386_lex_CFLAGS += $(subst -O3,-Wno-null-dereference,$(findstring -O3,$(CFLAGS)))
+
 i386_parse.o: i386_parse.c i386.mnemonics
 i386_lex.o: i386_parse.h
 i386_gendis_LDADD = $(libeu) -lm $(obstack_LIBS)
-- 
2.43.0

Reply via email to