peer review
- Make sure the repo structure follows the original lab, and that program compiles and works as intended. If not describe the problem in the issue. Tumme upp
- Local variables in registers for the subroutine "codgen" is allowed, but ALL local variables for the subroutine "decode" must be stored on the stack. If not describe the problem in the issue. Tumme upp
- Check the stack diagram for both subroutines, as in the "sum with frame pointer" example in "MIPS Programming". If not describe the problem in the issue. nja
- Check that the program provides sufficient high level comments in order to understand the implementation. If not describe the problem in the issue. tumme upp
Bra jobbat chefen, koden funkar och kommentarerna är tillräckliga. Du har ett stackdiagram på line 64 som gör mig osäker på om den ska där eller om den tillhör något och stacken i din ser ut att vara rätt beskriven men i "MIPS Programming" så är offseten negativa och i din kod är de positiva. Är inte hundra på om det är ett problem eller inte eftersom din kod funkar men om man ska jämföra med den stacken så är det inte helt hundra. Dina kommentarer kan kanske behöva liiite mer förtydligande, typ
00xx xxxx xxxx xxxx xxxx xxxx xxxn nnnn
0000 0000 0000 0000 0000 0000 0001 1111
#andi 0000 0000 0000 0000 0000 0000 000n nnnn (= n) Jag fattar dem men de kanske hade kunnat förtydligas med någon mening och kan du kan lägga till en kommentar när du returnar r i decode för att förtydliga att det sker i slutet av ELSE och THEN.