tenstorrent / tt-mlir

Tenstorrent MLIR compiler
https://tenstorrent.github.io/tt-mlir/
Apache License 2.0
71 stars 9 forks source link

TTNN atan2 op is reversing order of args #1113

Open kmitrovicTT opened 1 hour ago

kmitrovicTT commented 1 hour ago

Metal issue with detailed description: https://github.com/tenstorrent/tt-metal/issues/14470

This is blocking us from implementing StableHLO to TTIR conversion for atan2 OP https://github.com/tenstorrent/tt-mlir/pull/1056.

Op is not used in models as far as I can see from https://docs.google.com/spreadsheets/d/1HzHig2fJ4iKKoOZyxjZ3jddIHBEDDoop0MHJ-VkrOQY/edit?gid=0#gid=0.

Easy workaround would be to switch arguments during ttir -> ttnn conversion.

kmitrovicTT commented 1 hour ago

FYI @tt-mpantic