Fix generated cflags in LLVM/Clang for Star-MC3

[JonatanAntoni]

No description provided.

[JonatanAntoni]

@AlbertHuang-CPU please double check and confirm

[github-actions]

Test Results

  7 files   53 suites   6m 29s ⏱️
185 tests 168 ✅ 17 💤 0 ❌
692 runs  624 ✅ 68 💤 0 ❌

Results for commit 8cb32b3.

♻️ This comment has been updated with latest results.

[brondani]

LGTM

[AlbertHuang-CPU]

@AlbertHuang-CPU please double check and confirm

to align with llvm, we need to change Star-MC3 to be "star-mc3".
and I need more time to check other flags.

[jkrech]

@AlbertHuang-CPU do you have any updates?

[AlbertHuang-CPU]

I have checked the whole logic producing the flag combinations for star-mc3, and now can confirm no other error exist.
But we need to modify more places to lower case.
I don't have permission to make change on this branch.
@JonatanAntoni Please help to update the 4 "Star-MC3" to "star-mc3". (i.e. 4 of the 5, except the first one)
elseif(CPU STREQUAL "Star-MC3")
set(CLANG_ARCH "armv8.1m.main")
set(CPU_FEATURES "Star-MC3")
if(MVE STREQUAL "NO_MVE")
if(FPU STREQUAL "NO_FPU")
set(CPU_FEATURES "${CPU_FEATURES}+nomve+nofp")
elseif(FPU STREQUAL "SP_FPU")
set(CPU_FEATURES "${CPU_FEATURES}+nomve+nofp.dp")
elseif(FPU STREQUAL "DP_FPU")
set(CPU_FEATURES "${CPU_FEATURES}+nomve")
endif()
elseif(MVE STREQUAL "INT_MVE")
if(FPU STREQUAL "NO_FPU")
set(CPU_FEATURES "${CPU_FEATURES}+nomve.fp+nofp")
elseif(FPU STREQUAL "SP_FPU")
set(CPU_FEATURES "${CPU_FEATURES}+nomve.fp+nofp.dp")
elseif(FPU STREQUAL "DP_FPU")
message(FATAL_ERROR "Error: Star-MC3+nomve.fp is not supported!")
endif()
elseif(MVE STREQUAL "FP_MVE")
if(FPU STREQUAL "NO_FPU")
message(FATAL_ERROR "Error: Star-MC3+nofp is not supported!")
elseif(FPU STREQUAL "SP_FPU")
message(FATAL_ERROR "Error: Star-MC3+nofp.dp is not supported!")
endif()
endif()

[JonatanAntoni]

@AlbertHuang-CPU, you can always add comments and change suggestions to affected lines of code via the review function. I'll update the casing now.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.39%. Comparing base (3908cdb) to head (8cb32b3).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2341      +/-   ##
==========================================
- Coverage   64.43%   64.39%   -0.05%     
==========================================
  Files         145      145              
  Lines       26007    25969      -38     
  Branches    15675    15642      -33     
==========================================
- Hits        16758    16722      -36     
+ Misses       7084     7083       -1     
+ Partials     2165     2164       -1     

Flag	Coverage Δ
buildmgr-cov	79.87% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 4 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.