Closed kmabeeTT closed 1 week ago
Looks like they got mistakenly removed due to https://github.com/tenstorrent/tt-mlir/pull/578/files#diff-cb5555bbe8903cbdc6d6db76786b0a73ee1f999fbd22fc3f660c4b487daf58bf removing --ttmetal-serialize-to-binary="output=%t.ttm" from .mlir files resulting in no .ttm generated, resulting in no TTMetal tests run.
--ttmetal-serialize-to-binary="output=%t.ttm"
Easy fix to update the 3x TTMetal files to generate .ttm using new syntax advertised in that PR, ex:
./build/bin/ttmlir-opt --ttir-load-system-desc="path=ttrt-artifacts/system_desc.ttsys" --ttir-implicit-device --ttir-allocate --convert-ttir-to-ttmetal test/ttmlir/Silicon/TTMetal/to_layout.mlir | ./build/bin/ttmlir-translate --ttmetal-to-flatbuffer
FYI @rpavlovicTT
Meged PR #603 , marking closed.
kmabeeTT
commented
1 week ago kmabeeTT
commented
1 week ago
Looks like they got mistakenly removed due to https://github.com/tenstorrent/tt-mlir/pull/578/files#diff-cb5555bbe8903cbdc6d6db76786b0a73ee1f999fbd22fc3f660c4b487daf58bf removing
--ttmetal-serialize-to-binary="output=%t.ttm"from .mlir files resulting in no .ttm generated, resulting in no TTMetal tests run.Easy fix to update the 3x TTMetal files to generate .ttm using new syntax advertised in that PR, ex:
FYI @rpavlovicTT