Skip to content

Commit

Permalink
Move device_command.hpp to tt_metal/impl and split .cpp (#18762)
Browse files Browse the repository at this point in the history
### Ticket
#18448

### Problem description
Big header with all impl in it.
Header not needed int metal api anymore (prior PRs removed the
dependency)

### What's changed
For now just splitting as is.
Template could go away to make this maintainable.

### Checklist
- [x] [All post
commit](https://github.com/tenstorrent/tt-metal/actions/runs/13713426427)
  • Loading branch information
ayerofieiev-tt authored Mar 7, 2025
1 parent 830a442 commit 7a36e1b
Show file tree
Hide file tree
Showing 13 changed files with 1,187 additions and 921 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@
//
// SPDX-License-Identifier: Apache-2.0

#include <tt-metalium/device_command.hpp>
#include "cq_commands.hpp"
#include "memcpy.hpp"
#include <gtest/gtest.h>
#include "tt_metal/impl/dispatch/device_command.hpp"
#include "tt_metal/impl/dispatch/device_command_calculator.hpp"

namespace tt::tt_metal {
Expand Down
912 changes: 0 additions & 912 deletions tt_metal/api/tt-metalium/device_command.hpp

This file was deleted.

2 changes: 1 addition & 1 deletion tt_metal/distributed/mesh_trace.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
#include <mesh_coord.hpp>
#include <mesh_trace.hpp>

#include <tt-metalium/device_command.hpp>
#include "tt_metal/impl/dispatch/device_command.hpp"
#include "tt_metal/distributed/mesh_workload_utils.hpp"
#include "tt_metal/impl/trace/dispatch.hpp"

Expand Down
2 changes: 1 addition & 1 deletion tt_metal/distributed/mesh_workload.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
#include <tt_metal.hpp>

#include <tt_metal/impl/program/program_command_sequence.hpp>
#include <tt-metalium/device_command.hpp>
#include "tt_metal/impl/dispatch/device_command.hpp"
#include "tt_metal/distributed/mesh_workload_utils.hpp"

namespace tt::tt_metal::distributed {
Expand Down
2 changes: 1 addition & 1 deletion tt_metal/distributed/mesh_workload_utils.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
#include <host_api.hpp>
#include <command_queue.hpp>

#include <tt-metalium/device_command.hpp>
#include "tt_metal/impl/dispatch/device_command.hpp"
#include "tt_metal/impl/program/dispatch.hpp"
#include "tt_metal/impl/dispatch/dispatch_query_manager.hpp"
#include "tt_metal/distributed/mesh_workload_utils.hpp"
Expand Down
1 change: 1 addition & 0 deletions tt_metal/impl/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ set(IMPL_SRC
${CMAKE_CURRENT_SOURCE_DIR}/program/program.cpp
${CMAKE_CURRENT_SOURCE_DIR}/program/dispatch.cpp
${CMAKE_CURRENT_SOURCE_DIR}/dispatch/debug_tools.cpp
${CMAKE_CURRENT_SOURCE_DIR}/dispatch/device_command.cpp
${CMAKE_CURRENT_SOURCE_DIR}/dispatch/host_runtime_commands.cpp
${CMAKE_CURRENT_SOURCE_DIR}/dispatch/device_command_calculator.cpp
${CMAKE_CURRENT_SOURCE_DIR}/dispatch/dispatch_query_manager.cpp
Expand Down
3 changes: 2 additions & 1 deletion tt_metal/impl/buffers/dispatch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,11 @@
//
// SPDX-License-Identifier: Apache-2.0

#include <device_command.hpp>
#include <device.hpp>
#include "assert.hpp"
#include "dispatch.hpp"
#include "tt_metal/impl/dispatch/device_command.hpp"

#include <tt-metalium/command_queue_interface.hpp>
#include <tt-metalium/dispatch_settings.hpp>

Expand Down
Loading

0 comments on commit 7a36e1b

Please sign in to comment.