diff mbox series

package.py: skip dwarfsrcfiles for Clang LTO static libraries

Message ID 20251228031953.4024552-1-mark.yang@lge.com
State Under Review
Headers show
Series package.py: skip dwarfsrcfiles for Clang LTO static libraries | expand

Commit Message

mark.yang Dec. 28, 2025, 3:19 a.m. UTC
From: "mark.yang" <mark.yang@lge.com>

When using Clang toolchain with LTO enabled, static libraries (.a) contain
LLVM bitcode objects instead of ELF objects. dwarfsrcfiles cannot process
these files and fails with "not a valid ELF file" error.

Rather than catching the error message, guard the dwarfsrcfiles call by
checking for the specific conditions: static library (using is_static_lib()
magic check) + clang toolchain + lto in DISTRO_FEATURES. When all conditions
are met, skip the call silently.

Signed-off-by: mark.yang <mark.yang@lge.com>
---
 meta/lib/oe/package.py | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Alexander Kanavin Dec. 29, 2025, 11:13 a.m. UTC | #1
Thanks, this should be good.

Alex

On Sun, 28 Dec 2025 at 04:20, mark.yang via lists.openembedded.org
<mark.yang=lge.com@lists.openembedded.org> wrote:
>
> From: "mark.yang" <mark.yang@lge.com>
>
> When using Clang toolchain with LTO enabled, static libraries (.a) contain
> LLVM bitcode objects instead of ELF objects. dwarfsrcfiles cannot process
> these files and fails with "not a valid ELF file" error.
>
> Rather than catching the error message, guard the dwarfsrcfiles call by
> checking for the specific conditions: static library (using is_static_lib()
> magic check) + clang toolchain + lto in DISTRO_FEATURES. When all conditions
> are met, skip the call silently.
>
> Signed-off-by: mark.yang <mark.yang@lge.com>
> ---
>  meta/lib/oe/package.py | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/meta/lib/oe/package.py b/meta/lib/oe/package.py
> index baaa0cba02..279cd567b3 100644
> --- a/meta/lib/oe/package.py
> +++ b/meta/lib/oe/package.py
> @@ -770,6 +770,14 @@ def parse_debugsources_from_dwarfsrcfiles_output(dwarfsrcfiles_output):
>      return debugfiles.keys()
>
>  def source_info(file, d, fatal=True):
> +    # Skip static libraries when using Clang toolchain with LTO enabled.
> +    # In this case, .a files contain LLVM bitcode instead of ELF objects,
> +    # and dwarfsrcfiles cannot process them.
> +    if is_static_lib(file):
> +        if d.getVar('TOOLCHAIN') == "clang" and bb.utils.contains('DISTRO_FEATURES', 'lto', True, False, d):
> +            bb.debug(1, "Skipping dwarfsrcfiles for Clang LTO archive: %s" % file)
> +            return []
> +
>      cmd = ["dwarfsrcfiles", file]
>      try:
>          output = subprocess.check_output(cmd, universal_newlines=True, stderr=subprocess.STDOUT)
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#228580): https://lists.openembedded.org/g/openembedded-core/message/228580
> Mute This Topic: https://lists.openembedded.org/mt/116968059/1686489
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta/lib/oe/package.py b/meta/lib/oe/package.py
index baaa0cba02..279cd567b3 100644
--- a/meta/lib/oe/package.py
+++ b/meta/lib/oe/package.py
@@ -770,6 +770,14 @@  def parse_debugsources_from_dwarfsrcfiles_output(dwarfsrcfiles_output):
     return debugfiles.keys()
 
 def source_info(file, d, fatal=True):
+    # Skip static libraries when using Clang toolchain with LTO enabled.
+    # In this case, .a files contain LLVM bitcode instead of ELF objects,
+    # and dwarfsrcfiles cannot process them.
+    if is_static_lib(file):
+        if d.getVar('TOOLCHAIN') == "clang" and bb.utils.contains('DISTRO_FEATURES', 'lto', True, False, d):
+            bb.debug(1, "Skipping dwarfsrcfiles for Clang LTO archive: %s" % file)
+            return []
+
     cmd = ["dwarfsrcfiles", file]
     try:
         output = subprocess.check_output(cmd, universal_newlines=True, stderr=subprocess.STDOUT)