Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions app/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,11 @@ config SCHED_CPU_MASK_PIN_ONLY
config SMP_BOOT_DELAY
default y if SMP

# Allow compiler to determine whether to generate FLIX instructions.
choice COMPILER_CODEGEN_VLIW
default COMPILER_CODEGEN_VLIW_AUTO if "$(ZEPHYR_TOOLCHAIN_VARIANT)" = "xt-clang"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

VLIW is a generic term which can be a project default (AUTO) state regardless of compiler, no need to specify xt-clang here

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm pretty sure GCC cant emit VLIW for xtensa so it will need some check for Cadence compilers like above.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"AUTO" != "enabled", it means compiler will decide (use default config)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"AUTO" != "enabled", it means compiler will decide (use default config)

ok, so you are confirming no impact for GCC here and Zephyr SDK i.e. I just want to confirm we wont be passing any --enable-flix=auto type arguments to GCC which it wont understand if we remove the compiler check.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is only a configuration option, not a value passed to the compiler command line. The "auto" value does not result in setting any compile options.
https://github.com/zephyrproject-rtos/zephyr/blob/b4108b53f6d7ca8c1da596c7130318e64e26eaf0/CMakeLists.txt#L562-L568

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dcpleung sounds like we have all got alignment now so I think good to update and make non draft ?

Copy link
Contributor Author

@dcpleung dcpleung Feb 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Separated out the west.yml change into its own commit. So should be good to go.

Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent indentation: The indentation uses spaces here, but the rest of the file uses tabs (see lines 2 and 5). For consistency with the existing code, this line should use a tab instead of spaces.

Suggested change
default COMPILER_CODEGEN_VLIW_AUTO if "$(ZEPHYR_TOOLCHAIN_VARIANT)" = "xt-clang"
default COMPILER_CODEGEN_VLIW_AUTO if "$(ZEPHYR_TOOLCHAIN_VARIANT)" = "xt-clang"

Copilot uses AI. Check for mistakes.
endchoice

source "Kconfig.zephyr"

if SOC_FAMILY_INTEL_ADSP
Expand Down
2 changes: 1 addition & 1 deletion west.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ manifest:

- name: zephyr
repo-path: zephyr
revision: c1a2b3be459d4f34d31ae54774fd57e96438d237
revision: 05a3f62f4456b25f1830eeb1a09c39e4a2a8bc39
remote: zephyrproject

# Import some projects listed in zephyr/west.yml@revision
Expand Down
Loading