请大家看一下这种代码会有什么样的风险？

hiloco · 发表于 2013-9-11 11:33:30

a的位宽是多少？
a <= a + 1'b1;
右边有被截断的风险吧

yangxiaoyong · 发表于 2013-9-11 16:22:48

回复 20# kingbirdmax

这个就是准确的啊~

yangxiaoyong · 发表于 2013-9-11 16:23:44

回复 21# hiloco

能具体解释一下吗？谢谢~

574920045 · 发表于 2013-9-11 16:48:58

凑热闹，看看到底风险在哪！

hiloco · 发表于 2013-9-11 17:53:56

本帖最后由 hiloco 于 2013-9-12 09:07 编辑

回复 23# yangxiaoyong

我也是囫囵吞枣，如果不慎误人子弟不要怪我不是问你a的位宽么，如果a的位宽大于1位, a + 1'b1的结果很可能是a[0] + 1'b1，然后再赋值给左边a，则高位自动补零
a[N:1] <={N{1'b0}};
a[0] <= ~a[0];

yadog · 发表于 2013-9-12 10:23:57

作为一个纯粹的受控计数器，没有任何问题，不管是仿真还是实现

可以问下你们头儿具体问题在哪里

yangxiaoyong · 发表于 2013-9-12 15:49:11

回复 25# hiloco

我觉得这个在所有的计数器中都不会有影响

yangxiaoyong · 发表于 2013-9-12 15:50:15

回复 26# yadog

我决定不修改，等所谓内部审查的结果出来我再来分享给大家讨论~

matafeiye · 发表于 2013-9-12 16:23:04

异步复位有没有同步到当前时钟域？否则会有风险。

gjb649666926 · 发表于 2013-9-12 17:24:49

可能是coding style不好吧，给你一个我写的参考一下，a_r就是a，我们这边一般都要求这么写，尽量不要在时序always块中加入运算逻辑，最多加一个使能信号（flag），一般逻辑运算都放在组合逻辑中完成。

assign a_p = a_r + 1'b1;
always @ (posedge clk or negedge rst_n)
begin
if(!rst_n)
a_r <= 'b0;
else if(flag)
a_r <= a_p;
end

[讨论] 请大家看一下这种代码会有什么样的风险？

站长推荐 /2