missing defined (__ARM_FEATURE_CMSE) in some cases

ARM-software / CMSIS_6

CMSIS version 6 (successor of CMSIS_5)

https://arm-software.github.io/CMSIS_6/

Apache License 2.0

170 stars 56 forks source link

missing defined (__ARM_FEATURE_CMSE) in some cases #171

Open CanastraRF opened 3 months ago

CanastraRF commented 3 months ago

Hi most check for ARM_FEATURE_CMSE are like this defined (ARM_FEATURE_CMSE) && (ARM_FEATURE_CMSE == 3U) but in some lines are only (ARM_FEATURE_CMSE < 3U) so my tools reporting, that __ARM_FEATURE_CMSE can be undefined and interpreting as 0

please add everywhere a defined (__ARM_FEATURE_CMSE)

compare cmsis_armclang.h with cmsis_gcc.h

JonatanAntoni commented 3 months ago

Hi @CanastraRF, thanks for raising this. Would you be able to provide the required changes as a pull-request to speed up things?

ghost commented 2 months ago

Hello @CanastraRF, can you please check PR #177 if this solves your issue?

CanastraRF commented 2 months ago

Hello I've checked PR https://github.com/ARM-software/CMSIS_6/pull/177 and for us it is a good solution