[fpc-devel] Enabling hardware floating point processor for cortex-m4f on embedded and freertos

[Michael Ring]

lm4f120: yes, it should be moved in the makefile. But let me dig a 
little big, perhaps there are more files that are wrong, will come back 
to you.

Michael


Am 20.02.21 um 21:51 schrieb Florian Klämpfl via fpc-devel:
> Am 20.02.21 um 10:35 schrieb Michael Ring via fpc-devel:
>> I have created an issue with this patch for ermbedded target, for 
>> unknown reason the 'end;' at the end of the patch for 
>> rtl/embedded/arm/cortexm4f_start.inc is missing, likely when I 
>> copy/pasted I lost that line. Sorry for this....
>
> I committed it.
>
>>
>> Will there in the future be a way to provide pull-requests via github 
>> (or does this already work??) I guess that would reduce the 
>> likelyhood of such a stupid error..
>
> Most likely yes, however, so far nobody really found the time for the 
> switch to git.
>
>>
>> And then I also found another small mishap in 
>> rtl/freertos/arm/cortexm4f_start.inc, there a 'not' was left over 
>> after you changed the logic for fpu detection.
>>
>> When Investigating I saw that lm4f120 is built as armv7m but is 
>> actually armv7em (plus it uses m4f include). Not sure if it is woth 
>> fixing a seven years old file, so I left things as they are.
>
> This doesn't matter imo, it should be fixed. So you mean just moving 
> it to another subarch in the makefile?
> _______________________________________________
> fpc-devel maillist  -  fpc-devel at lists.freepascal.org
> https://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-devel