tenstorrent / tt-metal

:metal: TT-NN operator library, and TT-Metalium low level kernel programming model.
Apache License 2.0
429 stars 58 forks source link

Generalize the temp fix to layernorm in configuring the data types of in0 and in1 #1625

Closed mywoodstock closed 8 months ago

mywoodstock commented 1 year ago
commit a863b4bd0bf103df9b37b529f5213c801cff35df (HEAD -> asarje/bmm-mixed-prec-and-fixes-3, origin/asarje/bmm-mixed-prec-and-fixes-3)
Author: Abhinav Sarje <asarje@tenstorrent.com>
Date:   Wed Jul 12 23:03:19 2023 +0000

    #1625: Temporary fix to configure the unpacker correctly with and without in1

The temporary fix is:


    #ifdef FUSE_PRE_ADD
        binary_op_init_common(tt::CB::c_in0, tt::CB::c_in1);
    #else
        binary_op_init_common(tt::CB::c_in0, tt::CB::c_in0);
    #endif
mywoodstock commented 1 year ago

fyi @davorchap

davorchap commented 8 months ago

This as been fixed by @yugaoTT