* [U-Boot] [PATCH v2 1/2] tools, config.mk: add binutils-version
@ 2012-07-10 17:00 Allen Martin
2012-07-10 17:00 ` [U-Boot] [PATCH v2 2/2] arm: work around assembler bug Allen Martin
0 siblings, 1 reply; 7+ messages in thread
From: Allen Martin @ 2012-07-10 17:00 UTC (permalink / raw)
To: u-boot
Modeled after gcc-version, add function to get binutils version.
Signed-off-by: Allen Martin <amartin@nvidia.com>
---
config.mk | 1 +
tools/binutils-version.sh | 20 ++++++++++++++++++++
2 files changed, 21 insertions(+)
create mode 100755 tools/binutils-version.sh
diff --git a/config.mk b/config.mk
index c239f23..fa5e626 100644
--- a/config.mk
+++ b/config.mk
@@ -128,6 +128,7 @@ endif
# cc-version
# Usage gcc-ver := $(call cc-version)
cc-version = $(shell $(SHELL) $(SRCTREE)/tools/gcc-version.sh $(CC))
+binutils-version = $(shell $(SHELL) $(SRCTREE)/tools/binutils-version.sh $(AS))
#
# Include the make variables (CC, etc...)
diff --git a/tools/binutils-version.sh b/tools/binutils-version.sh
new file mode 100755
index 0000000..d4d9eb4
--- /dev/null
+++ b/tools/binutils-version.sh
@@ -0,0 +1,20 @@
+#!/bin/sh
+#
+# binutils-version [-p] gas-command
+#
+# Prints the binutils version of `gas-command' in a canonical 4-digit form
+# such as `0222' for binutils 2.22
+#
+
+gas="$*"
+
+if [ ${#gas} -eq 0 ]; then
+ echo "Error: No assembler specified."
+ printf "Usage:\n\t$0 <gas-command>\n"
+ exit 1
+fi
+
+MAJOR=$($gas --version | head -1 | awk '{print $NF}' | cut -d . -f 1)
+MINOR=$($gas --version | head -1 | awk '{print $NF}' | cut -d . -f 2)
+
+printf "%02d%02d\\n" $MAJOR $MINOR
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug
2012-07-10 17:00 [U-Boot] [PATCH v2 1/2] tools, config.mk: add binutils-version Allen Martin
@ 2012-07-10 17:00 ` Allen Martin
2012-07-10 17:25 ` Tom Rini
0 siblings, 1 reply; 7+ messages in thread
From: Allen Martin @ 2012-07-10 17:00 UTC (permalink / raw)
To: u-boot
Disable sibling call optimization based on binutils version. This is
to work around a bug in the assember in binutils versions < 2.22.
Branches to weak symbols can be incorrectly optimized in thumb mode to
a short branch (b.n instruction) that won't reach when the symbol gets
preempted.
http://sourceware.org/bugzilla/show_bug.cgi?id=12532
Signed-off-by: Allen Martin <amartin@nvidia.com>
---
changes for v2:
-changed GAS_BUG_12532 from yes/no to y/n to be consistent
-added additional warning about code size increase
---
arch/arm/config.mk | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
diff --git a/arch/arm/config.mk b/arch/arm/config.mk
index 3f4453a..bcabf17 100644
--- a/arch/arm/config.mk
+++ b/arch/arm/config.mk
@@ -87,3 +87,24 @@ endif
ifndef CONFIG_NAND_SPL
LDFLAGS_u-boot += -pie
endif
+
+#
+# binutils versions < 2.22 have a bug in the assembler where branches
+# to weak symbols can be incorrectly optimized in thumb mode to a
+# short branch (b.n instruction) that won't reach when the symbol
+# gets preempted
+#
+# http://sourceware.org/bugzilla/show_bug.cgi?id=12532
+#
+ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
+ifeq ($(GAS_BUG_12532),)
+export GAS_BUG_12532:=$(shell if [ $(call binutils-version) -lt 0222 ] ; then echo y; else echo n; fi)
+ifeq ($(GAS_BUG_12532),y)
+$(warning *** disabling sibling call optimzation because binutils version < 2.22)
+$(warning *** code size may be slightly larger)
+endif
+endif
+ifeq ($(GAS_BUG_12532),y)
+PLATFORM_RELFLAGS += -fno-optimize-sibling-calls
+endif
+endif
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug
2012-07-10 17:00 ` [U-Boot] [PATCH v2 2/2] arm: work around assembler bug Allen Martin
@ 2012-07-10 17:25 ` Tom Rini
2012-07-10 17:44 ` Allen Martin
0 siblings, 1 reply; 7+ messages in thread
From: Tom Rini @ 2012-07-10 17:25 UTC (permalink / raw)
To: u-boot
On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote:
> Disable sibling call optimization based on binutils version. This is
> to work around a bug in the assember in binutils versions < 2.22.
> Branches to weak symbols can be incorrectly optimized in thumb mode to
> a short branch (b.n instruction) that won't reach when the symbol gets
> preempted.
>
> http://sourceware.org/bugzilla/show_bug.cgi?id=12532
>
> Signed-off-by: Allen Martin <amartin@nvidia.com>
> ---
> changes for v2:
> -changed GAS_BUG_12532 from yes/no to y/n to be consistent
> -added additional warning about code size increase
> ---
> arch/arm/config.mk | 21 +++++++++++++++++++++
> 1 file changed, 21 insertions(+)
>
> diff --git a/arch/arm/config.mk b/arch/arm/config.mk
> index 3f4453a..bcabf17 100644
> --- a/arch/arm/config.mk
> +++ b/arch/arm/config.mk
> @@ -87,3 +87,24 @@ endif
> ifndef CONFIG_NAND_SPL
> LDFLAGS_u-boot += -pie
> endif
> +
> +#
> +# binutils versions < 2.22 have a bug in the assembler where branches
> +# to weak symbols can be incorrectly optimized in thumb mode to a
> +# short branch (b.n instruction) that won't reach when the symbol
> +# gets preempted
> +#
> +# http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> +#
> +ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> +ifeq ($(GAS_BUG_12532),)
> +export GAS_BUG_12532:=$(shell if [ $(call binutils-version) -lt 0222 ] ; then echo y; else echo n; fi)
Please don't squash out whitespace, just break this out into a
multi-line test ala the others in the toplevel config.mk.
> +ifeq ($(GAS_BUG_12532),y)
> +$(warning *** disabling sibling call optimzation because binutils version < 2.22)
> +$(warning *** code size may be slightly larger)
> +endif
> +endif
> +ifeq ($(GAS_BUG_12532),y)
> +PLATFORM_RELFLAGS += -fno-optimize-sibling-calls
> +endif
> +endif
This seems a bit longer than needed. Just check that it's y once and do
both things. Also, I _think_ we should be able to squish this down to:
ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \
then echo y; else echo n; fi)
ifeq ($(GAS_BUG_125320),y)
$(warning ...)
$(warning ...)
PLATFORM_RELFLAGS += ...
endif
endif
But I haven't tested it.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120710/2f841858/attachment.pgp>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug
2012-07-10 17:25 ` Tom Rini
@ 2012-07-10 17:44 ` Allen Martin
2012-07-10 17:50 ` Tom Rini
0 siblings, 1 reply; 7+ messages in thread
From: Allen Martin @ 2012-07-10 17:44 UTC (permalink / raw)
To: u-boot
On Tue, Jul 10, 2012 at 10:25:05AM -0700, Tom Rini wrote:
> * PGP Signed by an unknown key
>
> On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote:
> > Disable sibling call optimization based on binutils version. This is
> > to work around a bug in the assember in binutils versions < 2.22.
> > Branches to weak symbols can be incorrectly optimized in thumb mode to
> > a short branch (b.n instruction) that won't reach when the symbol gets
> > preempted.
> >
> > http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> >
> > Signed-off-by: Allen Martin <amartin@nvidia.com>
> > ---
> > changes for v2:
> > -changed GAS_BUG_12532 from yes/no to y/n to be consistent
> > -added additional warning about code size increase
> > ---
> > arch/arm/config.mk | 21 +++++++++++++++++++++
> > 1 file changed, 21 insertions(+)
> >
> > diff --git a/arch/arm/config.mk b/arch/arm/config.mk
> > index 3f4453a..bcabf17 100644
> > --- a/arch/arm/config.mk
> > +++ b/arch/arm/config.mk
> > @@ -87,3 +87,24 @@ endif
> > ifndef CONFIG_NAND_SPL
> > LDFLAGS_u-boot += -pie
> > endif
> > +
> > +#
> > +# binutils versions < 2.22 have a bug in the assembler where branches
> > +# to weak symbols can be incorrectly optimized in thumb mode to a
> > +# short branch (b.n instruction) that won't reach when the symbol
> > +# gets preempted
> > +#
> > +# http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> > +#
> > +ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> > +ifeq ($(GAS_BUG_12532),)
> > +export GAS_BUG_12532:=$(shell if [ $(call binutils-version) -lt 0222 ] ; then echo y; else echo n; fi)
>
> Please don't squash out whitespace, just break this out into a
> multi-line test ala the others in the toplevel config.mk.
Ok
> > +ifeq ($(GAS_BUG_12532),y)
> > +$(warning *** disabling sibling call optimzation because binutils version < 2.22)
> > +$(warning *** code size may be slightly larger)
> > +endif
> > +endif
> > +ifeq ($(GAS_BUG_12532),y)
> > +PLATFORM_RELFLAGS += -fno-optimize-sibling-calls
> > +endif
> > +endif
>
> This seems a bit longer than needed. Just check that it's y once and do
> both things. Also, I _think_ we should be able to squish this down to:
> ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \
> then echo y; else echo n; fi)
> ifeq ($(GAS_BUG_125320),y)
> $(warning ...)
> $(warning ...)
> PLATFORM_RELFLAGS += ...
> endif
> endif
>
> But I haven't tested it.
I wrote it the way I did because I only want the expensive
shell/binutils-version to get evaluated once per "make". I think what
you proposed would cause it to get re-evaluated every time the build
goes down into a subdir.
-Allen
--
nvpublic
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug
2012-07-10 17:44 ` Allen Martin
@ 2012-07-10 17:50 ` Tom Rini
2012-07-10 18:04 ` Allen Martin
0 siblings, 1 reply; 7+ messages in thread
From: Tom Rini @ 2012-07-10 17:50 UTC (permalink / raw)
To: u-boot
On Tue, Jul 10, 2012 at 10:44:33AM -0700, Allen Martin wrote:
> On Tue, Jul 10, 2012 at 10:25:05AM -0700, Tom Rini wrote:
> > * PGP Signed by an unknown key
> >
> > On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote:
> > > Disable sibling call optimization based on binutils version. This is
> > > to work around a bug in the assember in binutils versions < 2.22.
> > > Branches to weak symbols can be incorrectly optimized in thumb mode to
> > > a short branch (b.n instruction) that won't reach when the symbol gets
> > > preempted.
> > >
> > > http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> > >
> > > Signed-off-by: Allen Martin <amartin@nvidia.com>
> > > ---
> > > changes for v2:
> > > -changed GAS_BUG_12532 from yes/no to y/n to be consistent
> > > -added additional warning about code size increase
> > > ---
> > > arch/arm/config.mk | 21 +++++++++++++++++++++
> > > 1 file changed, 21 insertions(+)
> > >
> > > diff --git a/arch/arm/config.mk b/arch/arm/config.mk
> > > index 3f4453a..bcabf17 100644
> > > --- a/arch/arm/config.mk
> > > +++ b/arch/arm/config.mk
> > > @@ -87,3 +87,24 @@ endif
> > > ifndef CONFIG_NAND_SPL
> > > LDFLAGS_u-boot += -pie
> > > endif
> > > +
> > > +#
> > > +# binutils versions < 2.22 have a bug in the assembler where branches
> > > +# to weak symbols can be incorrectly optimized in thumb mode to a
> > > +# short branch (b.n instruction) that won't reach when the symbol
> > > +# gets preempted
> > > +#
> > > +# http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> > > +#
> > > +ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> > > +ifeq ($(GAS_BUG_12532),)
> > > +export GAS_BUG_12532:=$(shell if [ $(call binutils-version) -lt 0222 ] ; then echo y; else echo n; fi)
> >
> > Please don't squash out whitespace, just break this out into a
> > multi-line test ala the others in the toplevel config.mk.
>
> Ok
>
> > > +ifeq ($(GAS_BUG_12532),y)
> > > +$(warning *** disabling sibling call optimzation because binutils version < 2.22)
> > > +$(warning *** code size may be slightly larger)
> > > +endif
> > > +endif
> > > +ifeq ($(GAS_BUG_12532),y)
> > > +PLATFORM_RELFLAGS += -fno-optimize-sibling-calls
> > > +endif
> > > +endif
> >
> > This seems a bit longer than needed. Just check that it's y once and do
> > both things. Also, I _think_ we should be able to squish this down to:
> > ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> > GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \
> > then echo y; else echo n; fi)
> > ifeq ($(GAS_BUG_125320),y)
> > $(warning ...)
> > $(warning ...)
> > PLATFORM_RELFLAGS += ...
> > endif
> > endif
> >
> > But I haven't tested it.
>
> I wrote it the way I did because I only want the expensive
> shell/binutils-version to get evaluated once per "make". I think what
> you proposed would cause it to get re-evaluated every time the build
> goes down into a subdir.
It should still get evaluated once, I think. Kluding the
binutils-version script to echo 1 >> /tmp/test every time it's run would
tell tho for sure :)
--
Tom
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug
2012-07-10 17:50 ` Tom Rini
@ 2012-07-10 18:04 ` Allen Martin
2012-07-10 18:11 ` Tom Rini
0 siblings, 1 reply; 7+ messages in thread
From: Allen Martin @ 2012-07-10 18:04 UTC (permalink / raw)
To: u-boot
On Tue, Jul 10, 2012 at 10:50:34AM -0700, Tom Rini wrote:
> On Tue, Jul 10, 2012 at 10:44:33AM -0700, Allen Martin wrote:
> > On Tue, Jul 10, 2012 at 10:25:05AM -0700, Tom Rini wrote:
> > > * PGP Signed by an unknown key
> > >
> > > On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote:
> > > > Disable sibling call optimization based on binutils version. This is
> > > > to work around a bug in the assember in binutils versions < 2.22.
> > > > Branches to weak symbols can be incorrectly optimized in thumb mode to
> > > > a short branch (b.n instruction) that won't reach when the symbol gets
> > > > preempted.
> > > >
> > > > http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> > > >
> > > > Signed-off-by: Allen Martin <amartin@nvidia.com>
> > > > ---
> > > > changes for v2:
> > > > -changed GAS_BUG_12532 from yes/no to y/n to be consistent
> > > > -added additional warning about code size increase
> > > > ---
> > > > arch/arm/config.mk | 21 +++++++++++++++++++++
> > > > 1 file changed, 21 insertions(+)
> > > >
> > > > diff --git a/arch/arm/config.mk b/arch/arm/config.mk
> > > > index 3f4453a..bcabf17 100644
> > > > --- a/arch/arm/config.mk
> > > > +++ b/arch/arm/config.mk
> > > > @@ -87,3 +87,24 @@ endif
> > > > ifndef CONFIG_NAND_SPL
> > > > LDFLAGS_u-boot += -pie
> > > > endif
> > > > +
> > > > +#
> > > > +# binutils versions < 2.22 have a bug in the assembler where branches
> > > > +# to weak symbols can be incorrectly optimized in thumb mode to a
> > > > +# short branch (b.n instruction) that won't reach when the symbol
> > > > +# gets preempted
> > > > +#
> > > > +# http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> > > > +#
> > > > +ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> > > > +ifeq ($(GAS_BUG_12532),)
> > > > +export GAS_BUG_12532:=$(shell if [ $(call binutils-version) -lt 0222 ] ; then echo y; else echo n; fi)
> > >
> > > Please don't squash out whitespace, just break this out into a
> > > multi-line test ala the others in the toplevel config.mk.
> >
> > Ok
> >
> > > > +ifeq ($(GAS_BUG_12532),y)
> > > > +$(warning *** disabling sibling call optimzation because binutils version < 2.22)
> > > > +$(warning *** code size may be slightly larger)
> > > > +endif
> > > > +endif
> > > > +ifeq ($(GAS_BUG_12532),y)
> > > > +PLATFORM_RELFLAGS += -fno-optimize-sibling-calls
> > > > +endif
> > > > +endif
> > >
> > > This seems a bit longer than needed. Just check that it's y once and do
> > > both things. Also, I _think_ we should be able to squish this down to:
> > > ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> > > GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \
> > > then echo y; else echo n; fi)
> > > ifeq ($(GAS_BUG_125320),y)
> > > $(warning ...)
> > > $(warning ...)
> > > PLATFORM_RELFLAGS += ...
> > > endif
> > > endif
> > >
> > > But I haven't tested it.
> >
> > I wrote it the way I did because I only want the expensive
> > shell/binutils-version to get evaluated once per "make". I think what
> > you proposed would cause it to get re-evaluated every time the build
> > goes down into a subdir.
>
> It should still get evaluated once, I think. Kluding the
> binutils-version script to echo 1 >> /tmp/test every time it's run would
> tell tho for sure :)
I tried it out, it's obvious from the $(warning) it's getting
reevaluated:
$ ./MAKEALL -s tegra20
/home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
/home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
Configuring for harmony board...
text data bss dec hex filename
170334 4284 278352 452970 6e96a ./u-boot
11676 161 36 11873 2e61 ./spl/u-boot-spl
/home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
/home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
/home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
/home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
/home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
/home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
/home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
/home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
/home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
/home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
/home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
/home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
... and on and on
It makes sense since unless GAS_BUG_125320 is "export"ed to the new
make, it's going to be unset again each the build starts a new
"make -C"
-Allen
--
nvpublic
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v2 2/2] arm: work around assembler bug
2012-07-10 18:04 ` Allen Martin
@ 2012-07-10 18:11 ` Tom Rini
0 siblings, 0 replies; 7+ messages in thread
From: Tom Rini @ 2012-07-10 18:11 UTC (permalink / raw)
To: u-boot
On Tue, Jul 10, 2012 at 11:04:50AM -0700, Allen Martin wrote:
> On Tue, Jul 10, 2012 at 10:50:34AM -0700, Tom Rini wrote:
> > On Tue, Jul 10, 2012 at 10:44:33AM -0700, Allen Martin wrote:
> > > On Tue, Jul 10, 2012 at 10:25:05AM -0700, Tom Rini wrote:
> > > > * PGP Signed by an unknown key
> > > >
> > > > On Tue, Jul 10, 2012 at 10:00:17AM -0700, Allen Martin wrote:
> > > > > Disable sibling call optimization based on binutils version. This is
> > > > > to work around a bug in the assember in binutils versions < 2.22.
> > > > > Branches to weak symbols can be incorrectly optimized in thumb mode to
> > > > > a short branch (b.n instruction) that won't reach when the symbol gets
> > > > > preempted.
> > > > >
> > > > > http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> > > > >
> > > > > Signed-off-by: Allen Martin <amartin@nvidia.com>
> > > > > ---
> > > > > changes for v2:
> > > > > -changed GAS_BUG_12532 from yes/no to y/n to be consistent
> > > > > -added additional warning about code size increase
> > > > > ---
> > > > > arch/arm/config.mk | 21 +++++++++++++++++++++
> > > > > 1 file changed, 21 insertions(+)
> > > > >
> > > > > diff --git a/arch/arm/config.mk b/arch/arm/config.mk
> > > > > index 3f4453a..bcabf17 100644
> > > > > --- a/arch/arm/config.mk
> > > > > +++ b/arch/arm/config.mk
> > > > > @@ -87,3 +87,24 @@ endif
> > > > > ifndef CONFIG_NAND_SPL
> > > > > LDFLAGS_u-boot += -pie
> > > > > endif
> > > > > +
> > > > > +#
> > > > > +# binutils versions < 2.22 have a bug in the assembler where branches
> > > > > +# to weak symbols can be incorrectly optimized in thumb mode to a
> > > > > +# short branch (b.n instruction) that won't reach when the symbol
> > > > > +# gets preempted
> > > > > +#
> > > > > +# http://sourceware.org/bugzilla/show_bug.cgi?id=12532
> > > > > +#
> > > > > +ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> > > > > +ifeq ($(GAS_BUG_12532),)
> > > > > +export GAS_BUG_12532:=$(shell if [ $(call binutils-version) -lt 0222 ] ; then echo y; else echo n; fi)
> > > >
> > > > Please don't squash out whitespace, just break this out into a
> > > > multi-line test ala the others in the toplevel config.mk.
> > >
> > > Ok
> > >
> > > > > +ifeq ($(GAS_BUG_12532),y)
> > > > > +$(warning *** disabling sibling call optimzation because binutils version < 2.22)
> > > > > +$(warning *** code size may be slightly larger)
> > > > > +endif
> > > > > +endif
> > > > > +ifeq ($(GAS_BUG_12532),y)
> > > > > +PLATFORM_RELFLAGS += -fno-optimize-sibling-calls
> > > > > +endif
> > > > > +endif
> > > >
> > > > This seems a bit longer than needed. Just check that it's y once and do
> > > > both things. Also, I _think_ we should be able to squish this down to:
> > > > ifeq ($(CONFIG_SYS_THUMB_BUILD),y)
> > > > GAS_BUG_12532 := $(shell if [ $(call binutils-version) -lt 0222 ] ; \
> > > > then echo y; else echo n; fi)
> > > > ifeq ($(GAS_BUG_125320),y)
> > > > $(warning ...)
> > > > $(warning ...)
> > > > PLATFORM_RELFLAGS += ...
> > > > endif
> > > > endif
> > > >
> > > > But I haven't tested it.
> > >
> > > I wrote it the way I did because I only want the expensive
> > > shell/binutils-version to get evaluated once per "make". I think what
> > > you proposed would cause it to get re-evaluated every time the build
> > > goes down into a subdir.
> >
> > It should still get evaluated once, I think. Kluding the
> > binutils-version script to echo 1 >> /tmp/test every time it's run would
> > tell tho for sure :)
>
> I tried it out, it's obvious from the $(warning) it's getting
> reevaluated:
>
> $ ./MAKEALL -s tegra20
> /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
> /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
> Configuring for harmony board...
> text data bss dec hex filename
> 170334 4284 278352 452970 6e96a ./u-boot
> 11676 161 36 11873 2e61 ./spl/u-boot-spl
> /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
> /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
> /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
> /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
> /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
> /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
> /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
> /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
> /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
> /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
> /home/arm/u-boot/arch/arm/config.mk:102: *** disabling sibling call optimzation because binutils version < 2.22
> /home/arm/u-boot/arch/arm/config.mk:103: *** code size may be slightly larger
> ... and on and on
>
>
> It makes sense since unless GAS_BUG_125320 is "export"ed to the new
> make, it's going to be unset again each the build starts a new
> "make -C"
Thanks for proving me wrong. Just the white-space / multi line thing
please, thanks :)
--
Tom
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-07-10 18:11 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-10 17:00 [U-Boot] [PATCH v2 1/2] tools, config.mk: add binutils-version Allen Martin
2012-07-10 17:00 ` [U-Boot] [PATCH v2 2/2] arm: work around assembler bug Allen Martin
2012-07-10 17:25 ` Tom Rini
2012-07-10 17:44 ` Allen Martin
2012-07-10 17:50 ` Tom Rini
2012-07-10 18:04 ` Allen Martin
2012-07-10 18:11 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.