Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Core: added define guard __ARM_FEATURE_CMSE (#171) #177

Merged
1 commit merged into from
Jun 20, 2024
Merged

Conversation

ghost
Copy link

@ghost ghost commented Jun 20, 2024

Check if macro __ARM_FEATURE_CMSE is defined before evaluating it's value

@ghost ghost requested a review from JonatanAntoni June 20, 2024 07:19
Copy link

github-actions bot commented Jun 20, 2024

Test Results

   240 files   -   132     240 suites   - 132   0s ⏱️ - 8m 43s
    56 tests +    7      54 ✅ +   10      2 💤  -     3  0 ❌ ±0 
12 948 runs   - 5 280  10 860 ✅  - 1 104  2 088 💤  - 4 176  0 ❌ ±0 

Results for commit 2e62c25. ± Comparison against base commit e34e38b.

This pull request removes 49 and adds 56 tests. Note that renamed tests count towards both.
CMSIS-Core.src ‑ apsr.c
CMSIS-Core.src ‑ basepri.c
CMSIS-Core.src ‑ bkpt.c
CMSIS-Core.src ‑ clrex.c
CMSIS-Core.src ‑ clz.c
CMSIS-Core.src ‑ control.c
CMSIS-Core.src ‑ cp15.c
CMSIS-Core.src ‑ cpsr.c
CMSIS-Core.src ‑ dmb.c
CMSIS-Core.src ‑ dsb.c
…
TC_CML1Cache_CleanDCacheByAddrWhileDisabled
TC_CML1Cache_EnDisableDCache
TC_CML1Cache_EnDisableICache
TC_CoreFunc_APSR
TC_CoreFunc_BASEPRI
TC_CoreFunc_Control
TC_CoreFunc_EnDisIRQ
TC_CoreFunc_EncDecIRQPrio
TC_CoreFunc_FAULTMASK
TC_CoreFunc_FPSCR
…
This pull request removes 5 skipped tests and adds 2 skipped tests. Note that renamed tests count towards both.
CMSIS-Core.src ‑ lda.c
CMSIS-Core.src ‑ ldaex.c
CMSIS-Core.src ‑ stl.c
CMSIS-Core.src ‑ stlex.c
CMSIS-Core.src ‑ systick.c
TC_CoreInstr_WFE
TC_CoreInstr_WFI

♻️ This comment has been updated with latest results.

@ghost ghost merged commit 641cb5e into main Jun 20, 2024
8 checks passed
@ghost ghost deleted the Guard__ARM_FEATURE_CMSE branch June 20, 2024 10:35
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant