search

ARM-software / CMSIS-DSP

CMSIS-DSP embedded compute library for Cortex-M and Cortex-A
https://arm-software.github.io/CMSIS-DSP
Apache License 2.0
454 stars 122 forks source link

Include/arm_math_types_f16.h: undefined __ARM_FEATURE_MVE #146

Closed matthewbstokes-zz closed 5 months ago

matthewbstokes-zz commented 5 months ago

Missing check that __ARM_FEATURE_MVE is defined before checking its value

Include/arm_math_types_f16.h:52

#if !(__ARM_FEATURE_MVE & 2)
  #if !defined(DISABLEFLOAT16)
    #if defined(__ARM_FP16_FORMAT_IEEE) || defined(__ARM_FP16_FORMAT_ALTERNATIVE)
      typedef __fp16 float16_t;
      #define ARM_FLOAT16_SUPPORTED
    #endif
  #endif
#else

Update to:

#if !defined(__ARM_FEATURE_MVE) || !(__ARM_FEATURE_MVE & 2)
christophe0606 commented 5 months ago

Corrected in latest commit