Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/libxslt: fix bfin compile
@ 2016-09-21 20:21 Jörg Krause
  2016-09-21 20:32 ` Peter Korsgaard
  0 siblings, 1 reply; 4+ messages in thread
From: Jörg Krause @ 2016-09-21 20:21 UTC (permalink / raw)
  To: buildroot

A GCC bug is triggered when the compiler is run with an optimization level
'-Os/-O2/-O3' for the Blackfin architecture:

```
error: unable to find a register to spill in class 'CCREGS'
```

Workaround this bug by forcing an optimization level of '-O1' when building
libxslt for the Blackfin architecture.

See GCC bug report:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77311

Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
---
 package/libxslt/libxslt.mk | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/package/libxslt/libxslt.mk b/package/libxslt/libxslt.mk
index 868ba6a..4253a60 100644
--- a/package/libxslt/libxslt.mk
+++ b/package/libxslt/libxslt.mk
@@ -19,6 +19,12 @@ LIBXSLT_CONF_OPTS = \
 LIBXSLT_CONFIG_SCRIPTS = xslt-config
 LIBXSLT_DEPENDENCIES = libxml2
 
+# GCC bug with Os/O2/O3, PR77311
+# error: unable to find a register to spill in class 'CCREGS'
+ifeq ($(BR2_bfin),y)
+LIBXSLT_CONF_ENV += CFLAGS="$(TARGET_CFLAGS) -O1"
+endif
+
 # If we have enabled libgcrypt then use it, else disable crypto support.
 ifeq ($(BR2_PACKAGE_LIBGCRYPT),y)
 LIBXSLT_DEPENDENCIES += libgcrypt
-- 
2.10.0

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] package/libxslt: fix bfin compile
  2016-09-21 20:21 [Buildroot] [PATCH] package/libxslt: fix bfin compile Jörg Krause
@ 2016-09-21 20:32 ` Peter Korsgaard
  2016-09-21 20:35   ` Jörg Krause
  0 siblings, 1 reply; 4+ messages in thread
From: Peter Korsgaard @ 2016-09-21 20:32 UTC (permalink / raw)
  To: buildroot

>>>>> "J?rg" == J?rg Krause <joerg.krause@embedded.rocks> writes:

 > A GCC bug is triggered when the compiler is run with an optimization level
 > '-Os/-O2/-O3' for the Blackfin architecture:

 > ```
 > error: unable to find a register to spill in class 'CCREGS'
 > ```

 > Workaround this bug by forcing an optimization level of '-O1' when building
 > libxslt for the Blackfin architecture.

 > See GCC bug report:
 > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77311

 > Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>

Committed after adding references to the autobuilder failures, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] package/libxslt: fix bfin compile
  2016-09-21 20:32 ` Peter Korsgaard
@ 2016-09-21 20:35   ` Jörg Krause
  2016-09-21 20:41     ` Peter Korsgaard
  0 siblings, 1 reply; 4+ messages in thread
From: Jörg Krause @ 2016-09-21 20:35 UTC (permalink / raw)
  To: buildroot

On Mi, 2016-09-21 at 22:32 +0200, Peter Korsgaard wrote:
> > 
> > > 
> > > > 
> > > > > 
> > > > > > 
> > > > > > "J?rg" == J?rg Krause <joerg.krause@embedded.rocks> writes:
> 
> ?> A GCC bug is triggered when the compiler is run with an
> optimization level
> ?> '-Os/-O2/-O3' for the Blackfin architecture:
> 
> ?> ```
> ?> error: unable to find a register to spill in class 'CCREGS'
> ?> ```
> 
> ?> Workaround this bug by forcing an optimization level of '-O1' when
> building
> ?> libxslt for the Blackfin architecture.
> 
> ?> See GCC bug report:
> ?> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77311
> 
> ?> Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks>
> 
> Committed after adding references to the autobuilder failures,
> thanks.

Oh, I did it in v2, too. Thanks!

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] package/libxslt: fix bfin compile
  2016-09-21 20:35   ` Jörg Krause
@ 2016-09-21 20:41     ` Peter Korsgaard
  0 siblings, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2016-09-21 20:41 UTC (permalink / raw)
  To: buildroot

>>>>> "J?rg" == J?rg Krause <joerg.krause@embedded.rocks> writes:

Hi,

 >> Committed after adding references to the autobuilder failures,
 >> thanks.

 > Oh, I did it in v2, too. Thanks!

Sorry, I hadn't noticed your v2 yet.

-- 
Venlig hilsen,
Peter Korsgaard 

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-09-21 20:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-21 20:21 [Buildroot] [PATCH] package/libxslt: fix bfin compile Jörg Krause
2016-09-21 20:32 ` Peter Korsgaard
2016-09-21 20:35   ` Jörg Krause
2016-09-21 20:41     ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox