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


Reply via email to