Skip to content

intel_adsp: Xtensa: Build failures on intel_adsp platforms with cadence clang toolchain #90383

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
gbarkadiusz opened this issue May 23, 2025 · 3 comments · May be fixed by #90702
Open

intel_adsp: Xtensa: Build failures on intel_adsp platforms with cadence clang toolchain #90383

gbarkadiusz opened this issue May 23, 2025 · 3 comments · May be fixed by #90702
Assignees
Labels
area: Xtensa Xtensa Architecture bug The issue is a bug, or the PR is fixing a bug platform: Intel ADSP Intel Audio platforms priority: medium Medium impact/importance bug Regression Something, which was working, does not anymore

Comments

@gbarkadiusz
Copy link
Collaborator

gbarkadiusz commented May 23, 2025

Describe the bug
Build Failures on: intel_adsp/ace15_mtpm/sim, intel_adsp/ace20_lnl/sim, intel_adsp/ace_30/ptl/sim.

To Reproduce
Checkout to latest upstream (b1f9351)
Run: twister -p intel_adsp/ace15_mptm/sim -T tests/

Impact
Affects ACE platforms

Logs and console output

In file included from <path_to>/modules/lib/picolibc/newlib/libc/stdlib/nano-free.c:29:
<path_to>/modules/lib/picolibc/newlib/libc/stdlib/nano-malloc.h:60:9: error: unknown type name 'max_align_t'
typedef max_align_t align_chunk_t;
        ^
1 error generated.

Environment:

  • OS: Linux
  • Toolchain xt-clang
  • Commit SHA or Version used (b1f9351)

Additional context
Bisect result: (843d661)

843d661b820e067f28112d2ea7dd822c2ecbcba2 is the first bad commit
commit 843d661b820e067f28112d2ea7dd822c2ecbcba2
Author: Keith Packard <keithp@keithp.com>
Date:   Thu Mar 27 15:03:06 2025 -0700

    modules: Update picolibc to version 1.8.10

    Switch to latest picolib release, version 1.8.10

    Signed-off-by: Keith Packard <keithp@keithp.com>

 west.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
@gbarkadiusz gbarkadiusz added bug The issue is a bug, or the PR is fixing a bug Regression Something, which was working, does not anymore area: Xtensa Xtensa Architecture labels May 23, 2025
@gbarkadiusz gbarkadiusz added the platform: Intel ADSP Intel Audio platforms label May 23, 2025
@nashif nashif changed the title ACE: Xtensa: Build failures on ACE platforms. intel_adsp: Xtensa: Build failures on intel_adsp platforms with cadence clang toolchain May 23, 2025
@dkalowsk dkalowsk added the priority: medium Medium impact/importance bug label May 27, 2025
@nashif
Copy link
Member

nashif commented May 27, 2025

@keith-packard any idea what is going on here?

@dcpleung
Copy link
Member

Part of the issue is that picolibc does not inherent --std= from Zephyr build system, and defaults to C11. And xt-clang defaults to using XCC's pre-C11 headers by default, where max_align_t is not defined.

@keith-packard
Copy link
Collaborator

lmk if you want me to review any picolibc patches; the whole cmake build bits for that are purely in support of Zephyr, so if there are useful changes to be made, feel free.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Xtensa Xtensa Architecture bug The issue is a bug, or the PR is fixing a bug platform: Intel ADSP Intel Audio platforms priority: medium Medium impact/importance bug Regression Something, which was working, does not anymore
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants