On 1/30/2018 11:40 PM, Thomas Monjalon wrote: > 16/12/2017 01:13, Ferruh Yigit: >> Add "debug" target to build library with debug symbols and optimization >> disabled. >> >> This is shortcut for exiting method to compile with >> EXTRA_CFLAGS="-O0 -g3" >> >> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> >> --- >> + debug build library with debug symbols > [...] >> +.PHONY: debug >> +debug: >> + $(Q)$(MAKE) EXTRA_CFLAGS="-O0 -g3" > > This target will override any EXTRA_CFLAGS, > so we cannot build in debug mode with more extra cflags this way.
How about EXTRA_CFLAGS+="-O0 -g3" which is not override EXTRA_CFLAGS but merge them, can be ok? > > I think the cons are higher than the pros. >
