On 04/30/19 13:07, Bob Feng wrote: > BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1764 > > Some compiler flags restrict the compiler from making > arbitrary decisions while handling undefined C/C++ behaviors. > Therefore they can be used to fix some issues caused by undefined behavior. > > For example, for GCC, the following flags are available: > -fno-delete-null-pointer-checks tells > the compiler NOT to assume that null pointer deference does not exist. > -fwrapv tells the compiler that signed overflow always wraps. > > This patch is going to add these 2 build options to > BaseTool GCC build option. > > Signed-off-by: Bob Feng <bob.c.f...@intel.com> > Cc: Liming Gao <liming....@intel.com> > --- > BaseTools/Source/C/Makefiles/header.makefile | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/BaseTools/Source/C/Makefiles/header.makefile > b/BaseTools/Source/C/Makefiles/header.makefile > index c2397b796c..eac295b4dd 100644 > --- a/BaseTools/Source/C/Makefiles/header.makefile > +++ b/BaseTools/Source/C/Makefiles/header.makefile > @@ -69,11 +69,12 @@ BUILD_OPTFLAGS = -O2 $(EXTRA_OPTFLAGS) > ifeq ($(DARWIN),Darwin) > # assume clang or clang compatible flags on OS X > BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror \ > -Wno-deprecated-declarations -Wno-self-assign -Wno-unused-result -nostdlib -g > else > -BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror \ > +BUILD_CFLAGS = -MD -fshort-wchar -fno-strict-aliasing -fwrapv \ > +-fno-delete-null-pointer-checks -Wall -Werror \ > -Wno-deprecated-declarations -Wno-stringop-truncation -Wno-restrict \ > -Wno-unused-result -nostdlib -g > endif > BUILD_LFLAGS = > BUILD_CXXFLAGS = -Wno-unused-result >
Reviewed-by: Laszlo Ersek <ler...@redhat.com> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#39859): https://edk2.groups.io/g/devel/message/39859 Mute This Topic: https://groups.io/mt/31418842/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-