xiaoxiang781216 commented on a change in pull request #5662: URL: https://github.com/apache/incubator-nuttx/pull/5662#discussion_r818262375
########## File path: arch/arm/Kconfig ########## @@ -908,6 +908,11 @@ config ARM_FP_ABI_SOFT ---help--- Pass float value via integer register (-mfloat-abi=softfp) +config ARM_FP_NEON Review comment: This config can't work, FPU and NEON share the same register set. OS save and restore this register set only when FPU is enabled. If you enable NEON but disable FPU, thread context will corrupt randomly. ARM also require that FP hardware must exist before silicon vender can enable NEON when they design SoC, so I don't believe there is any benefit to enable NEON but not FPU. -- 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