-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
irq_armv7m: missing fpu selection for Armcc5 #1266
Comments
Playing with the command line options I could not create FPv4-SP in the info {FPU} but using --fpu option I was able to see VFPv4_D16
|
Partly addressed in #1267 |
This would be the correct condition (taking {FPU} from the table above)?
|
@0xc0170, please check with @GorgonMeducer about getting this into a PR. Thanks! |
For FPU, use armasm to select fpu selection. This will be fixed upstream in the next version of CMSIS. Meanwhile, we use our local patch. Taken from --cpu selection for armasm: --cpu | {FPU} Cortex-M4.fp.sp | VFPv4_SP_D16 Cortex-M7.fp.sp | FPv5-SP Cortex-M7.fp.dp | FPv5_D16 Tracking issue: ARM-software/CMSIS_5#1266
For FPU, use armasm to select fpu selection. This will be fixed upstream in the next version of CMSIS. Meanwhile, we use our local patch. Taken from --cpu selection for armasm: --cpu | {FPU} Cortex-M4.fp.sp | VFPv4_SP_D16 Cortex-M7.fp.sp | FPv5-SP Cortex-M7.fp.dp | FPv5_D16 Tracking issue: ARM-software/CMSIS_5#1266
The file https://github.com/ARM-software/CMSIS_5/blob/develop/CMSIS/RTOS2/RTX/Source/ARM/irq_armv7m.s contains only one FPU selection:
There are multiple options of FPU for v7m as I know of. For instance I found it failing for cortex-m4f (using
--cpu Cortex-M4.fp.sp
).It should be (if I got the FPU flags correct):
The text was updated successfully, but these errors were encountered: