Intrinsic definition collision for ARMv7-A/Cortex-A9

[stellar-aria]

Compiler: GCC 13.2.1 (Arm GNU Toolchain)
Target: Cortex-A9 (-mcpu=cortex-a9 -mfpu=neon) bare metal, no OS
Flags: -DNEON

With the addition of #219, building now fails due to redefinition of certain intrinsics and macros previously defined at CMSIS/Core/Incclude/cmsis_gcc.h:938. There's a note at

CMSIS-DSP/Include/arm_math_types.h

Lines 115 to 116 in 4404d3e

	/* Intrinsincs are defined in CMSIS Core that is not available when
	* targeting Cortex-A or Cortex-R

that seems relevant, but if that were the case there wouldn't be a name collision.

Attempting to build without providing an inclusion against CMSIS/Core/Include instead raises a different error: "fatal error: cmsis_compiler.h: No such file or directory".

I assume that Core must have its includes available for the DSP module, but doing so creates the collision. Deleting the check against ARM_MATH_NEON in arm_math_types.h removes the issue.

[christophe0606]

@stellar-aria You can't build by just defining ARM_MATH_NEON and __GNUC_PYTHON__ ?

__GNUC_PYTHON__ will disable the inclusion of cmsis_gcc.h (and the dependency to CMSIS-Core) and ARM_MATH_NEON will enable the Neon variants when available.

[christophe0606]

Above flags are for make. With cmake it should be -DNEON=ON and -DHOST=ON

[christophe0606]

I have made a commit for this issue. It should solve the problem with CMSIS Core if you build with above defines.

[stellar-aria]

Yes, removing the CMSIS Core reference and adding HOST ON worked, thanks so much!