| Message ID | 20230711131105.19203-2-alx@kernel.org |
|---|---|
| State | New |
| Headers | show |
| Series | Use -fno-delete-null-pointer-checks to build glibc | expand |
On 11/07/23 10:11, Alejandro Colomar via Libc-alpha wrote: > After this change, patches will be more clear about what they change, as > they won't need to reflow lines. > > Signed-off-by: Alejandro Colomar <alx@kernel.org> I think this can be installed independently of the rest of the patchset. > --- > Makeconfig | 17 ++++++++++++----- > 1 file changed, 12 insertions(+), 5 deletions(-) > > diff --git a/Makeconfig b/Makeconfig > index 77d7fd14df..369a596e79 100644 > --- a/Makeconfig > +++ b/Makeconfig > @@ -1043,11 +1043,18 @@ CPPFLAGS = $(config-extra-cppflags) $(CPPFLAGS-config) \ > $(CPPFLAGS-$(<F)) $(CPPFLAGS-$(@F)) $(CPPFLAGS-$(basename $(@F))) \ > -DTOP_NAMESPACE=glibc > > -override CFLAGS = -std=gnu11 -fgnu89-inline $(config-extra-cflags) \ > - $(filter-out %frame-pointer,$(+cflags)) $(+gccwarn-c) \ > - $(+extra-math-flags) $(+extra-time-flags) \ > - $(sysdep-CFLAGS) $(CFLAGS-$(suffix $@)) $(CFLAGS-$(<F)) \ > - $(CFLAGS-$(@F)) $(tls-model) \ > +override CFLAGS = -std=gnu11 \ Maybe move '-std=gnu11' to next line as well. > + -fgnu89-inline \ > + $(config-extra-cflags) \ > + $(filter-out %frame-pointer,$(+cflags)) \ > + $(+gccwarn-c) \ > + $(+extra-math-flags) \ > + $(+extra-time-flags) \ > + $(sysdep-CFLAGS) \ > + $(CFLAGS-$(suffix $@)) \ > + $(CFLAGS-$(<F)) \ > + $(CFLAGS-$(@F)) \ > + $(tls-model) \ > $(foreach lib,$(libof-$(basename $(@F))) \ > $(libof-$(<F)) $(libof-$(@F)),$(CFLAGS-$(lib))) > # Use our copies of cstdlib and cmath.
diff --git a/Makeconfig b/Makeconfig index 77d7fd14df..369a596e79 100644 --- a/Makeconfig +++ b/Makeconfig @@ -1043,11 +1043,18 @@ CPPFLAGS = $(config-extra-cppflags) $(CPPFLAGS-config) \ $(CPPFLAGS-$(<F)) $(CPPFLAGS-$(@F)) $(CPPFLAGS-$(basename $(@F))) \ -DTOP_NAMESPACE=glibc -override CFLAGS = -std=gnu11 -fgnu89-inline $(config-extra-cflags) \ - $(filter-out %frame-pointer,$(+cflags)) $(+gccwarn-c) \ - $(+extra-math-flags) $(+extra-time-flags) \ - $(sysdep-CFLAGS) $(CFLAGS-$(suffix $@)) $(CFLAGS-$(<F)) \ - $(CFLAGS-$(@F)) $(tls-model) \ +override CFLAGS = -std=gnu11 \ + -fgnu89-inline \ + $(config-extra-cflags) \ + $(filter-out %frame-pointer,$(+cflags)) \ + $(+gccwarn-c) \ + $(+extra-math-flags) \ + $(+extra-time-flags) \ + $(sysdep-CFLAGS) \ + $(CFLAGS-$(suffix $@)) \ + $(CFLAGS-$(<F)) \ + $(CFLAGS-$(@F)) \ + $(tls-model) \ $(foreach lib,$(libof-$(basename $(@F))) \ $(libof-$(<F)) $(libof-$(@F)),$(CFLAGS-$(lib))) # Use our copies of cstdlib and cmath.
After this change, patches will be more clear about what they change, as they won't need to reflow lines. Signed-off-by: Alejandro Colomar <alx@kernel.org> --- Makeconfig | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-)