yf13 commented on code in PR #15800:
URL: https://github.com/apache/nuttx/pull/15800#discussion_r1950180149


##########
arch/arm/Kconfig:
##########
@@ -982,6 +982,7 @@ config ARCH_ARMV7R
        default n
        select ARCH_HAVE_CPUINFO
        select ARCH_HAVE_PERF_EVENTS
+       select ARM_THUMB

Review Comment:
   I don't know exactly, may be that affects the toolchain options and without 
it signal test fails. It seems to me that 
`arm/src/common/gnu/arm_signal_handler.S`  implicitly requires it now? I don't 
know ARM assembly thus didn't touch it.



-- 
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