-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Open
Labels
Community: QuestionFurther information is requestedFurther information is requested
Description
Description
I have a question. It seems that many other projects use the architecture name as the value for CMAKE_SYSTEM_PROCESSOR. For example, for Cortex-M0, it should be set to armv6m. However, if we directly set it to the CPU type now, we may encounter compatibility issues with some libraries:
nuttx/arch/arm/src/cmake/platform.cmake
Lines 25 to 31 in b37a527
| if(CONFIG_ARCH_CORTEXM0) | |
| set(CMAKE_SYSTEM_PROCESSOR cortex-m0) | |
| elseif(CONFIG_ARCH_CORTEXM3) | |
| set(CMAKE_SYSTEM_PROCESSOR cortex-m3) | |
| elseif(CONFIG_ARCH_CORTEXM4) | |
| set(CMAKE_SYSTEM_PROCESSOR cortex-m4) | |
| elseif(CONFIG_ARCH_CORTEXM7) |
For the 3rd party lib, such as iree and openamp
https://github.com/iree-org/iree/blob/6cdb40cf6ebe1eea457dc4bd5bbdb9d1a60ad5a0/build_tools/cmake/iree_macros.cmake#L49-L72
Should we modify the definition to this more commonly used form? Doing so could improve compatibility with other 3rd party libraries.
Verification
- I have verified before submitting the report.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Community: QuestionFurther information is requestedFurther information is requested