[fpc-devel] Trunk does not compile for armv6m since rev. 26346

Nico Erfurth ne at erfurth.eu
Thu Jan 2 16:46:34 CET 2014


On 02.01.14 14:35, Jonas Maebe wrote:
> 
> On 02 Jan 2014, at 14:11, Nico Erfurth wrote:
> 
>> I've added a new CPU-feature flag CPUARM_HAS_UMULL and also used it for
>> the 64x64=>64bit code.
> 
> I think it would be cleaner to also use it in
> tarmaddnode.use_generic_mul32to64 (maybe it's also necessary for
> correctness, since in pass_1 you check is_64bitint(left.resultdef) in
> tarmaddnode.pass_1 and that will return false if use_generic_mul32to64
> returned true, which it currently always does).

You're right and Florian already fixed it. I'll have to work on my arm
testsetup for the different architectures and ISAs to catch stuff like
this BEFORE i'm committing. ;)

Nico




More information about the fpc-devel mailing list