casaroli commented on code in PR #21363:
URL: https://github.com/apache/nuttx/pull/21363#discussion_r4453583603
##########
arch/arm/Kconfig:
##########
@@ -40,7 +40,6 @@ config ARM_TOOLCHAIN_GNU_OABI
config ARM_TOOLCHAIN_CLANG
bool "LLVM Clang toolchain"
- depends on !LIBM_TOOLCHAIN
Review Comment:
This dependency keeps Clang off configs that use the toolchain libm, and the
Clang libm in CI has no `__fpclassifyd` or `sinf`. Without it, the
`CONFIG_ARM_TOOLCHAIN_CLANG` lines of tools/ci/testlist build with Clang and 7
jobs of this PR fail, e.g. nucleo-f412zg:coremark `ld.lld: error: undefined
symbol: __fpclassifyd`.
Fix: drop this commit from the PR, or first fix the Clang libm link and the
Clang build errors that CI shows.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
