relaxed-simd icon indicating copy to clipboard operation
relaxed-simd copied to clipboard

Possible bug in spec of relaxeddotmul

Open rossberg opened this issue 1 year ago • 2 comments

The spec defines:

   \EXPROFDET & \relaxeddotmul_{M,N}(i_1, i_2) &=& [ \imul_N(\signed_M(i_1), i_2), \imul_N(\signed_M(i_1), \signed_M(i_2)) ] \\
   & \relaxeddotmul_{M,N}(i_1, i_2) &=& \imul_N(\extends_{M,N}(i_1), \extends_{M,N}(i_2)) \\

The use of signed in the relaxed case seems off. Should that be

   \EXPROFDET & \relaxeddotmul_{M,N}(i_1, i_2) &=& [ \imul_N(\extends_{M,N}(i_1), extendu_{M,N}), \imul_N(\extends_{M,N}(i_1), \extends_{M,N}(i_2)) ] \\

?

rossberg avatar Sep 03 '24 15:09 rossberg

~In fact, aren't the additions missing as well?~

rossberg avatar Sep 04 '24 09:09 rossberg

Ohyea, should be extend to make the bitwidth line up. Thanks!

ngzhian avatar Sep 05 '24 03:09 ngzhian