All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] gcc: Dropping mips workaround
@ 2021-11-15 11:23 Richard Purdie
  2021-11-15 17:19 ` [OE-core] " Khem Raj
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Purdie @ 2021-11-15 11:23 UTC (permalink / raw)
  To: openembedded-core

I've tested without this and the ptest results for mips are the same with
and without it so the issue this was fixing in gcc 9 was likely resolved
by gcc 11.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/recipes-devtools/binutils/binutils_2.37.bb | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/meta/recipes-devtools/binutils/binutils_2.37.bb b/meta/recipes-devtools/binutils/binutils_2.37.bb
index 7430bf13425..12a6fb55776 100644
--- a/meta/recipes-devtools/binutils/binutils_2.37.bb
+++ b/meta/recipes-devtools/binutils/binutils_2.37.bb
@@ -27,10 +27,6 @@ EXTRA_OECONF:class-native = "--enable-targets=all \
 
 PACKAGECONFIG ??= "${@bb.utils.filter('DISTRO_FEATURES', 'debuginfod', d)}"
 PACKAGECONFIG[debuginfod] = "--with-debuginfod, --without-debuginfod, elfutils"
-# gcc9.0 end up mis-compiling libbfd.so with O2 which then crashes on target
-# So remove -O2 and use -Os as workaround
-SELECTED_OPTIMIZATION:remove:mipsarch = "-O2"
-SELECTED_OPTIMIZATION:append:mipsarch = " -Os"
 
 do_install:class-native () {
 	autotools_do_install
-- 
2.32.0



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

* Re: [OE-core] [PATCH] gcc: Dropping mips workaround
  2021-11-15 11:23 [PATCH] gcc: Dropping mips workaround Richard Purdie
@ 2021-11-15 17:19 ` Khem Raj
  0 siblings, 0 replies; 2+ messages in thread
From: Khem Raj @ 2021-11-15 17:19 UTC (permalink / raw)
  To: Richard Purdie, openembedded-core

lgtm

On 11/15/21 3:23 AM, Richard Purdie wrote:
> I've tested without this and the ptest results for mips are the same with
> and without it so the issue this was fixing in gcc 9 was likely resolved
> by gcc 11.
> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>   meta/recipes-devtools/binutils/binutils_2.37.bb | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/meta/recipes-devtools/binutils/binutils_2.37.bb b/meta/recipes-devtools/binutils/binutils_2.37.bb
> index 7430bf13425..12a6fb55776 100644
> --- a/meta/recipes-devtools/binutils/binutils_2.37.bb
> +++ b/meta/recipes-devtools/binutils/binutils_2.37.bb
> @@ -27,10 +27,6 @@ EXTRA_OECONF:class-native = "--enable-targets=all \
>   
>   PACKAGECONFIG ??= "${@bb.utils.filter('DISTRO_FEATURES', 'debuginfod', d)}"
>   PACKAGECONFIG[debuginfod] = "--with-debuginfod, --without-debuginfod, elfutils"
> -# gcc9.0 end up mis-compiling libbfd.so with O2 which then crashes on target
> -# So remove -O2 and use -Os as workaround
> -SELECTED_OPTIMIZATION:remove:mipsarch = "-O2"
> -SELECTED_OPTIMIZATION:append:mipsarch = " -Os"
>   
>   do_install:class-native () {
>   	autotools_do_install
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#158279): https://lists.openembedded.org/g/openembedded-core/message/158279
> Mute This Topic: https://lists.openembedded.org/mt/87066754/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


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

end of thread, other threads:[~2021-11-15 17:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-15 11:23 [PATCH] gcc: Dropping mips workaround Richard Purdie
2021-11-15 17:19 ` [OE-core] " Khem Raj

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.