Skip to contentSkip to conversation

x86: minor Makefile simplification through use of cc-ifversion

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

Subsystems (2)
lore
Share & tools
lore

Frans Pop

author
#0 lore raw

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

---

Change was suggested by Sam Ravnborg; see http://lkml.org/lkml/2009/7/18/15.
Tested to work correctly.

arch/x86/Makefile +2 -2
index 1b68659..1f3851a 100644
@@ -32,8 +32,8 @@ ifeq ($(CONFIG_X86_32),y)
 
         # 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:
-        KBUILD_CFLAGS += $(shell if [ $(call cc-version) -lt 0400 ] ; then \
-                echo $(call cc-option,-fno-unit-at-a-time); fi ;)
+        KBUILD_CFLAGS += $(call cc-ifversion, -lt, 0400, \
+				$(call cc-option,-fno-unit-at-a-time))
 
         # CPU-specific tuning. Anything which can be shared with UML should go here.
         include $(srctree)/arch/x86/Makefile_32.cpu
Reviewed-by on v1 1/1
#2 lore raw
In reply to Frans Pop
Quoted 5 lines · Frans Pop

---

Change was suggested by Sam Ravnborg; see http://lkml.org/lkml/2009/7/18/15.
Tested to work correctly.

Quoted 21 lines
arch/x86/Makefile +2 -3
index 1b68659..1f3851a 100644
@@ -32,8 +32,8 @@ ifeq ($(CONFIG_X86_32),y)
 
        # 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:
-        KBUILD_CFLAGS += $(shell if [ $(call cc-version) -lt 0400 ] ; then \
-                echo $(call cc-option,-fno-unit-at-a-time); fi ;)
+        KBUILD_CFLAGS += $(call cc-ifversion, -lt, 0400, \
+				$(call cc-option,-fno-unit-at-a-time))
 
        # CPU-specific tuning. Anything which can be shared with UML should go here.
        include $(srctree)/arch/x86/Makefile_32.cpu
--

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

#3 lore raw

[tip:x86/kbuild] x86: Simplify the Makefile in a minor way through use of cc-ifversion

In reply to Frans Pop
Commit-ID:  c7bd0414d681706a32105895cae20fb9090db52e
Gitweb:     http://git.kernel.org/tip/c7bd0414d681706a32105895cae20fb9090db52e
Author:     Frans Pop <[email protected]>
AuthorDate: Thu, 23 Jul 2009 20:56:27 +0200
Committer:  Ingo Molnar <[email protected]>
CommitDate: Tue, 4 Aug 2009 14:32:24 +0200

x86: Simplify the Makefile in a minor way through use of cc-ifversion

LKML-Reference: <[email protected]>

---
 arch/x86/Makefile |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
arch/x86/Makefile +2 -2
index 1b68659..1f3851a 100644
@@ -32,8 +32,8 @@ ifeq ($(CONFIG_X86_32),y)
 
         # 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:
-        KBUILD_CFLAGS += $(shell if [ $(call cc-version) -lt 0400 ] ; then \
-                echo $(call cc-option,-fno-unit-at-a-time); fi ;)
+        KBUILD_CFLAGS += $(call cc-ifversion, -lt, 0400, \
+				$(call cc-option,-fno-unit-at-a-time))
 
         # CPU-specific tuning. Anything which can be shared with UML should go here.
         include $(srctree)/arch/x86/Makefile_32.cpu