pkarashchenko commented on a change in pull request #5662:
URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r818393922
##########
File path: arch/arm/Kconfig
##########
@@ -901,6 +901,13 @@ config ARM_HAVE_NEON
---help---
Decide whether support NEON instruction
+config ARM_FP_ABI_SOFT
Review comment:
> Emm, but it's a bug of old toolchain, I think in this case we should
suggest user to update their toolchain instead of do some magic in correct code
to make toolchain happy.
>
> And softfp is a valid option (disabled by default), it can make
Toolchain.defs more general (if some one want to use softfp, just enable this
option instead of modifiy the board level Make.defs)
From my practice toolchain update is not always a straight forward process
and migration to a new toolchain take a lot of time and efforts. So I'm fine if
we leave `softfp` as a fallback option for those who still are not ready to
change toolchain withing a single day.
--
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: commits-unsubscr...@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
