2

我在将 C 代码转换为组合函数 (nCr) 的 MIPS 汇编代码时遇到问题。

nCr = (n-1Cr-1) + (n-1Cr)

当我将 int 5 用于 n 并将 3 用于 r （数字数据）时，结果必须为 10。

我想使用递归和堆栈指针，但堆栈溢出时出现错误。

我在下面有我的 MIPS 代码。

我的代码有什么问题？

我无法很好地识别问题...

##data
.data
digit: .word 5, 3

.text
.globl main

main:
##load data
la $t0, digit
lw $a0, 0($t0)  #put 5 in a
lw $a1, 4($t0)  #put 3 in b

##call Function comb
jal comb
##save return value in $t1
move $t1, $v0

##print result
li $v0, 1
add $a0, $0, $t1
syscall

##exit
li $v0, 10
syscall

##Function int comb(int a, int b)
comb:
addi $sp, $sp, -8
sw $ra, 4($sp)
sw $s0, 0($sp)

##base case
bne $a0, $a1, gen  #if (a==b)
addi $v0, $0, 1 #$v0 (1)
j rtn
bne $a1, $0, gen  #if (b==0)
addi $v0, $0, 1 #$v0 (1)
j rtn

##recursive call
gen:
addi $a0, $a0, -1 #$a0 (a-1)
addi $a1, $a1, -1 #$a1 (b-1)
jal comb  #call comb(a-1, b-1)
add $s0, $v0, $0  #$s0 comb(a-1, b-1)
addi $a1, $a1, 1  #$a1 (b)
jal comb  #call comb(a-1, b)
add $v0, $v0, $s0 #$v0 (comb(a-1, b-1) + comb(a-1, b))
j rtn

rtn:
lw $s0, 0($sp)
lw $ra, 4($sp)
addi $sp, $sp, 8
jr $ra

.end
4

2 回答 2

1

您的代码存在多个问题。

首先，您选择的递归关系非常低效，而且比必要的复杂得多。甚至维基百科也有几个更好的重复关系，例如

n over k = (n-1 over k-1) * (n/k)

避免多次递归到函数中（从而允许函数以尾递归方式编写）。

您正在使用的重复有一个额外的缺点，您必须同时检查k==0 和 k==n。

这将我们带到您的实施中，该实施未能正确检查这两个条件：

##base case
bne $a0, $a1, gen  #if (a==b)
addi $v0, $0, 1 #$v0 (1)
j rtn
bne $a1, $0, gen  #if (b==0)
addi $v0, $0, 1 #$v0 (1)
j rtn

这些测试中的第一个会跳过第二个测试 if a!=b，无论是否b==0，因此第二个测试是不可访问的。您必须将代码更改为

##base case
bne $a0, $a1, isbzero  #if (a==b)
addi $v0, $0, 1 #$v0 (1)
j rtn
isbzero:
bne $a1, $0, gen  #if (b==0)
addi $v0, $0, 1 #$v0 (1)
j rtn

最后，您不会在递归调用之前保留函数参数的值。如果您想符合 ABI，则不能假设函数参数寄存器中的值在调用过程中被保留。

特别是之后

##recursive call
gen:
addi $a0, $a0, -1 #$a0 (a-1)
addi $a1, $a1, -1 #$a1 (b-1)
jal comb  #call comb(a-1, b-1)

以下

add $s0, $v0, $0  #$s0 comb(a-1, b-1)
addi $a1, $a1, 1  #$a1 (b)
jal comb  #call comb(a-1, b)

$a0和的值不正确$a1。

如果 ABI 合规性对您来说并不重要，您可以在返回之前通过再次增加参数来恢复这些值。

于 2018-10-10T20:49:40.677 回答
0

您的基本问题是您没有在（递归）调用中保存寄存器值，因此这些值被破坏了。$a0 和 $a1 是调用者保存寄存器，所以任何调用都可能破坏它们，而你的函数comb实际上破坏了它们。但这意味着在第一次递归调用之后，值就消失了，所以第二次递归调用是垃圾。

您需要在第一次递归调用之前将 $a0 和 $a1 的值保存在堆栈帧中，并在它返回后（在第二次调用之前）恢复它们。您还需要在第二次调用前后保存 $v0 的值，因为 $v0 类似地被调用破坏了。

于 2018-10-10T20:46:24.257 回答