[buildmgr] AC6 options for Cortex-M85

[jeromecoutant]

Hi

I try to compare
https://developer.arm.com/documentation/ka005067/latest/
with
https://github.com/Open-CMSIS-Pack/devtools/blob/main/tools/buildmgr/cbuildgen/config/AC6.6.16.2.cmake#L197-L211

=> "--cpu=Cortex-M85.no_mvefp" looks not correct ?

and with
https://github.com/Open-CMSIS-Pack/devtools/blob/main/tools/buildmgr/cbuildgen/config/AC6.6.16.2.cmake#L447-L462

=> should we use lower case ?
=> "-mcpu=Cortex-M85+nomve.fp -mfloat-abi=hard" looks not correct ?

Thx

[ReinhardKeil]

Unfortunately Arm Compiler 6.16.2 does not support Cortex-M85.
See the list here https://developer.arm.com/documentation/107873/latest/ - under 1.2

The 6.22.1 FuSa version covers Cortex-M85 support
https://developer.arm.com/documentation/109905/062201/Release-overview/Product-description

[jeromecoutant]

M85 was introduced in #305 with 6.18 version
File was renamed into 6.16 later...

[jeromecoutant]

The 6.22.1 FuSa version covers Cortex-M85 support https://developer.arm.com/documentation/109905/062201/Release-overview/Product-description

And options in AC6.6.16.2.cmake M85 options are correct for that?

[ReinhardKeil]

OK, I get your point. The issue is that we support with one make file all AC6 compiler versions.

[ReinhardKeil]

@jeromecoutant do you actually get an error? I believe the mcpu option is not case senstive as we compile a lot for Cortex-M85 and did not see any problem.

[jeromecoutant]

do you actually get an error?

For now yes... But I don't find any clear docs for mcpu options...
And
if(MVE STREQUAL "MVE") set(ARMCLANG_CPU "-mcpu=Cortex-M85+nomve.fp
seems wrong, nomve sould be removed?

we compile a lot for Cortex-M85 and did not see any problem.

Note Windows is not very case sensitive...

[brondani]

@jeromecoutant All CPUs in this file have had historically capitalized names, the OS should be irrelevant in this case.
Please share the error you are getting and possibly the conditions to reproduce it.

If we stricly attain to the mentioned table indeed there is no entry for "Integer MVE included + FP MVE not included + Scalar FP included" which would correspond to -mcpu=Cortex-M85+nomve.fp, so we should rather return "Error: CPU is not supported!", is that your expectation?