[ASM & Hex✓] Fire Red: What is wrong with this ASM code

[Entaro Ceraphenine]

Hi folks!
Can someone tell me what is wrong with the following ASM code? The thumb batch compiler is not compiling it for some reason.
The code:

Spoiler:

.text
.align 2
.thumb
.thumb_func

main:
push {r0-r2, lr}
ldr r2, =(0x0300553C) @ r2 = address of year
ldrh r0, [r2] @ r0 = current year
mov r1, 0xFF @ r1 = 255
add r1, r1, #0x6E @ r1 = 365
mul r0, r0, r1 @ r0 = year * 365
pop {r0-r2, pc}

.align 2

This is just one part of another huge ASM and so far the other parts of the code doesn't seem to have any problems.

Additional question:
Where is the play time stored in Fire Red RAM?
And is there any in game function that calculates how many days it has been since the start of the game?

 

[Entaro Ceraphenine]

Is there any other way I can accomplish this? ;-;

 

[mgriffin]

Sorry, I was mistaken about your code, I'm not sure why it doesn't work :(

But it's probably possible to work it out by looking through GBAtek's Thumb documentation.

 

[Entaro Ceraphenine]

Alright! I'll take a look at the documentation! Thank you :D

 

[Noodlewhiz]

It's not compiling because line 4 in the function should be mov r1, #0xFF (your forgot the #). Additionally, even if you call this function from another function, the return value will be incorrect, as you discard the computed value at the end by popping r0. I recommend removing the line push {r0-r2, lr}, and replacing the last line in the function with bx lr.

 

[Entaro Ceraphenine]

It's not compiling because line 4 in the function should be mov r1, #0xFF (your forgot the #). Additionally, even if you call this function from another function, the return value will be incorrect, as you discard the computed value at the end by popping r0. I recommend removing the line push {r0-r2, lr}, and replacing the last line in the function with bx lr.

My bad.. did not notice the missing #.. thanks a lot!

And this routine isn't complete yet. I'll put the final answers of my result in some variables before popping the registers! But I will keep bx lr in mind for future functions! Thank you! :D