search

tenstorrent / tt-metal

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

Remove llrt.cpp #2618

Open abhullar-tt opened 1 year ago

abhullar-tt commented 1 year ago

A lot of llrt APIs are difficult to follow and unnecessary wrappers around tt_cluster / UMD APIs

I think there is general consensus about doing this

pgkeller commented 1 year ago

I agree that most of llrt.cpp should go away, what's not clear to me is if llrt should go away or actually get added to. As an abstraction level it makes sense, I think cluster sits in there. If you meant llrt.cpp, should the subject line change?