[meta-python][PATCH] python-lxml: replace -Og with -O for mips64-32

[meta-python][PATCH] python-lxml: replace -Og with -O for mips64-32

[mingli.yu]

From: Mingli Yu <Mingli.Yu@windriver.com>

With below logic in local.conf:
MACHINE ??= "qemumips64"
MULTILIB_GLOBAL_VARIANTS_append = " libn32"
MULTILIBS ?= "multilib:lib32 multilib:libn32"
DEFAULTTUNE_virtclass-multilib-lib32 ?= "mips"
DEFAULTTUNE_virtclass-multilib-libn32 ?= "mips64-n32"
require conf/multilib.conf
DEBUG_BUILD = "1"
$ bitbake lib32-python-lxml
[snip]
| {standard input}: Assembler messages:
| {standard input}:1488805: Error: branch out of range
| Compile failed: command 'mips-pokymllib32-linux-gcc' failed with exit status 1
| creating tmp
| cc -I/yocto/builds//tmp/work/mips-pokymllib32-linux/lib32-python-lxml/4.3.4-r0/lib32-recipe-sysroot/usr/include/libxml2 -I/usr/include/libxml2 -c /tmp/xmlXPathInitPZiWuM.c -o tmp/xmlXPathInitPZiWuM.o
| unable to execute 'cc': No such file or directory
[snip]

When DEBUG_BUILD enabled, -Og pass to compiler,
replace -Og with -O to fix the above compile error.

Signed-off-by: Mingli Yu <Mingli.Yu@windriver.com>
---
 meta-python/recipes-devtools/python/python-lxml.inc | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/meta-python/recipes-devtools/python/python-lxml.inc b/meta-python/recipes-devtools/python/python-lxml.inc
index d67c17883..dd73a412b 100644
--- a/meta-python/recipes-devtools/python/python-lxml.inc
+++ b/meta-python/recipes-devtools/python/python-lxml.inc
@@ -33,6 +33,18 @@ DISTUTILS_INSTALL_ARGS += " \
 
 inherit pypi
 
+# {standard input}: Assembler messages:
+# {standard input}:1488805: Error: branch out of range
+DEBUG_OPTIMIZATION_remove_mips = " -Og"
+DEBUG_OPTIMIZATION_append_mips = " -O"
+BUILD_OPTIMIZATION_remove_mips = " -Og"
+BUILD_OPTIMIZATION_append_mips = " -O"
+
+DEBUG_OPTIMIZATION_remove_mipsel = " -Og"
+DEBUG_OPTIMIZATION_append_mipsel = " -O"
+BUILD_OPTIMIZATION_remove_mipsel = " -Og"
+BUILD_OPTIMIZATION_append_mipsel = " -O"
+
 do_configure_prepend() {
     sed -i -e 's/--version/--modversion/' ${B}/setupinfo.py
 }
-- 
2.21.0

Re: [meta-python][PATCH] python-lxml: replace -Og with -O for mips64-32

[Khem Raj]

On Wed, Jun 19, 2019 at 11:50 PM <mingli.yu@windriver.com> wrote:
>
> From: Mingli Yu <Mingli.Yu@windriver.com>
>
> With below logic in local.conf:
> MACHINE ??= "qemumips64"
> MULTILIB_GLOBAL_VARIANTS_append = " libn32"
> MULTILIBS ?= "multilib:lib32 multilib:libn32"
> DEFAULTTUNE_virtclass-multilib-lib32 ?= "mips"
> DEFAULTTUNE_virtclass-multilib-libn32 ?= "mips64-n32"
> require conf/multilib.conf
> DEBUG_BUILD = "1"
> $ bitbake lib32-python-lxml
> [snip]
> | {standard input}: Assembler messages:
> | {standard input}:1488805: Error: branch out of range
> | Compile failed: command 'mips-pokymllib32-linux-gcc' failed with exit status 1
> | creating tmp
> | cc -I/yocto/builds//tmp/work/mips-pokymllib32-linux/lib32-python-lxml/4.3.4-r0/lib32-recipe-sysroot/usr/include/libxml2 -I/usr/include/libxml2 -c /tmp/xmlXPathInitPZiWuM.c -o tmp/xmlXPathInitPZiWuM.o
> | unable to execute 'cc': No such file or directory
> [snip]
>
> When DEBUG_BUILD enabled, -Og pass to compiler,
> replace -Og with -O to fix the above compile error.
>
> Signed-off-by: Mingli Yu <Mingli.Yu@windriver.com>
> ---
>  meta-python/recipes-devtools/python/python-lxml.inc | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/meta-python/recipes-devtools/python/python-lxml.inc b/meta-python/recipes-devtools/python/python-lxml.inc
> index d67c17883..dd73a412b 100644
> --- a/meta-python/recipes-devtools/python/python-lxml.inc
> +++ b/meta-python/recipes-devtools/python/python-lxml.inc
> @@ -33,6 +33,18 @@ DISTUTILS_INSTALL_ARGS += " \
>
>  inherit pypi
>
> +# {standard input}: Assembler messages:
> +# {standard input}:1488805: Error: branch out of range
> +DEBUG_OPTIMIZATION_remove_mips = " -Og"
> +DEBUG_OPTIMIZATION_append_mips = " -O"
> +BUILD_OPTIMIZATION_remove_mips = " -Og"
> +BUILD_OPTIMIZATION_append_mips = " -O"
> +
> +DEBUG_OPTIMIZATION_remove_mipsel = " -Og"
> +DEBUG_OPTIMIZATION_append_mipsel = " -O"
> +BUILD_OPTIMIZATION_remove_mipsel = " -Og"
> +BUILD_OPTIMIZATION_append_mipsel = " -O"
> +

this is a workaround and probably ok in meanwhile. However can you
generaste a preprocessed file
where it breaks the assembly. So we can report it upstream

>  do_configure_prepend() {
>      sed -i -e 's/--version/--modversion/' ${B}/setupinfo.py
>  }
> --
> 2.21.0
>