Except that you could get a tight code, I don't see anything wrong.
Oops!! However after my answers to other yours question I see: Put the instruction MOV A, @R1 instead MOV A, R1
Looking at the way you place labels you follow a flow chart. Which is not bad at all.

Regards,
Silviu