Skip to contentSkip to conversation

um: minor Makefile simplification through use of cc-ifversion

PATCH In review Frans Pop 3 participants 4 messages Jul 23, 2009 – Nov 3, 2009

Subsystems (2)
lore
Share & tools
lore

Frans Pop

author
#0 lore raw

PATCH 1/1 um: minor Makefile simplification through use of cc-ifversion

---

Change was suggested by Sam Ravnborg; see http://lkml.org/lkml/2009/7/18/15.
Untested, but a similar patch for x86 was tested to work.

arch/um/Makefile-i386 +4 -3
index 302cbe5..2c3e658 100644
@@ -35,8 +35,9 @@ cflags-y += -ffreestanding
 # Disable unit-at-a-time mode on pre-gcc-4.0 compilers, it makes gcc use
 # a lot more stack due to the lack of sharing of stacklots.  Also, gcc
 # 4.3.0 needs -funit-at-a-time for extern inline functions.
-KBUILD_CFLAGS += $(shell if [ $(call cc-version) -lt 0400 ] ; then \
-			echo $(call cc-option,-fno-unit-at-a-time); \
-			else echo $(call cc-option,-funit-at-a-time); fi ;)
+KBUILD_CFLAGS += $(call cc-ifversion, -lt, 0400, \
+			$(call cc-option,-fno-unit-at-a-time))
+KBUILD_CFLAGS += $(call cc-ifversion, -ge, 0400, \
+			$(call cc-option,-funit-at-a-time))
 
 KBUILD_CFLAGS += $(cflags-y)
author
#3 lore raw
In reply to Frans Pop

Hi Sam,

You planned to take any of these patches not picked up by arch maintainers in kbuild.git. This looks to be the only one not yet included in mainline.

Cheers,
FJP

Quoted 24 lines · Frans Pop

---

Change was suggested by Sam Ravnborg; see
http://lkml.org/lkml/2009/7/18/15. Untested, but a similar patch for x86 was tested to work.

arch/um/Makefile-i386 +0 -0
index 302cbe5..2c3e658 100644
@@ -35,8 +35,9 @@ cflags-y += -ffreestanding
 # Disable unit-at-a-time mode on pre-gcc-4.0 compilers, it makes gcc
 use # a lot more stack due to the lack of sharing of stacklots.  Also,

gcc # 4.3.0 needs -funit-at-a-time for extern inline functions.

arch/um/Makefile-i386 +4 -3
-KBUILD_CFLAGS += $(shell if [ $(call cc-version) -lt 0400 ] ; then \
-			echo $(call cc-option,-fno-unit-at-a-time); \
-			else echo $(call cc-option,-funit-at-a-time); fi ;)
+KBUILD_CFLAGS += $(call cc-ifversion, -lt, 0400, \
+			$(call cc-option,-fno-unit-at-a-time))
+KBUILD_CFLAGS += $(call cc-ifversion, -ge, 0400, \
+			$(call cc-option,-funit-at-a-time))
 
 KBUILD_CFLAGS += $(cflags-y)