diff options
author | Viet-Hoa Do <viet-hoa.do@arm.com> | 2023-07-20 17:31:47 +0100 |
---|---|---|
committer | Viet-Hoa Do <viet-hoa.do@arm.com> | 2023-07-21 13:33:12 +0000 |
commit | 25d26f4d86042e0ca52ac1bef4039b187f77b5b3 (patch) | |
tree | 9a4ee7d5173883a2c67a68689d591efd746f2b5f /compute_kernel_writer/src | |
parent | 8dfb8820d5fe0f72a923eccc3bb73ee0b87d5511 (diff) | |
download | ComputeLibrary-25d26f4d86042e0ca52ac1bef4039b187f77b5b3.tar.gz |
Change TileOperand to a view of a tile object
* TileOperand instead of being the tile object now is only a view
of a tile object.
- declare_tile now returns a TileOperand object, not a reference
to a TileOperand object.
- This is to prepare for the posibility that the users need
to perform operations on part of a tile (e.g. a scalar value,
a vector, a sub-tile).
Partially resolves: COMPMID-6391
Signed-off-by: Viet-Hoa Do <viet-hoa.do@arm.com>
Change-Id: I35c08b22a384a756d99dcd04cbe66fc57bd548d2
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/9958
Reviewed-by: Gunes Bayir <gunes.bayir@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Benchmark: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'compute_kernel_writer/src')
-rw-r--r-- | compute_kernel_writer/src/KernelWriter.cpp | 17 | ||||
-rw-r--r-- | compute_kernel_writer/src/TileOperand.cpp | 9 | ||||
-rw-r--r-- | compute_kernel_writer/src/cl/CLKernelWriter.cpp | 22 | ||||
-rw-r--r-- | compute_kernel_writer/src/cl/CLKernelWriter.h | 11 | ||||
-rw-r--r-- | compute_kernel_writer/src/cl/CLTile.h | 13 |
5 files changed, 39 insertions, 33 deletions
diff --git a/compute_kernel_writer/src/KernelWriter.cpp b/compute_kernel_writer/src/KernelWriter.cpp index 347ae5b545..7b83eade6f 100644 --- a/compute_kernel_writer/src/KernelWriter.cpp +++ b/compute_kernel_writer/src/KernelWriter.cpp @@ -22,14 +22,13 @@ * SOFTWARE. */ +#include "ckw/KernelWriter.h" #include "ckw/Error.h" #include "ckw/TileOperand.h" -#include "ckw/KernelWriter.h" #include "ckw/types/TargetArchitecture.h" #include "ckw/types/TargetLanguage.h" #include "src/cl/CLKernelWriter.h" -#include <iterator> namespace ckw { @@ -55,15 +54,19 @@ int32_t KernelWriter::id_space() const return _id_space; } -TileOperand &KernelWriter::add_operand(std::unique_ptr<TileOperand> &operand_ptr) +std::string KernelWriter::generate_full_name(const std::string &name) const { - auto it = _operands.insert(std::move(operand_ptr)); - return *it.first->get(); + return "G" + std::to_string(id_space()) + "__" + name; } -std::string KernelWriter::generate_full_name(const std::string &name) const +TileOperand KernelWriter::create_tile_operand(ITile &tile) { - return "G" + std::to_string(id_space()) + "__" + name; + return TileOperand(tile); +} + +ITile &KernelWriter::get_tile(const TileOperand &operand) +{ + return operand._tile; } } // namespace ckw diff --git a/compute_kernel_writer/src/TileOperand.cpp b/compute_kernel_writer/src/TileOperand.cpp index 83914b3216..7d180feec8 100644 --- a/compute_kernel_writer/src/TileOperand.cpp +++ b/compute_kernel_writer/src/TileOperand.cpp @@ -26,5 +26,10 @@ namespace ckw { - TileOperand::~TileOperand() = default; -}
\ No newline at end of file + +TileOperand::TileOperand(ITile &tile) + : _tile(tile) +{ +} + +} // namespace ckw diff --git a/compute_kernel_writer/src/cl/CLKernelWriter.cpp b/compute_kernel_writer/src/cl/CLKernelWriter.cpp index 2f8b1c95ce..bc056c67a2 100644 --- a/compute_kernel_writer/src/cl/CLKernelWriter.cpp +++ b/compute_kernel_writer/src/cl/CLKernelWriter.cpp @@ -24,14 +24,15 @@ #include "src/cl/CLKernelWriter.h" #include "ckw/Error.h" -#include "src/cl/CLTile.h" +#include "ckw/TileOperand.h" #include "src/cl/CLHelpers.h" +#include "src/cl/CLTile.h" #include <cstdint> namespace ckw { -CLKernelWriter::CLKernelWriter() = default; +CLKernelWriter::CLKernelWriter() = default; CLKernelWriter::~CLKernelWriter() = default; std::unique_ptr<Kernel> CLKernelWriter::emit_kernel(const std::string &name) @@ -61,12 +62,12 @@ const std::string &CLKernelWriter::body_source_code() const return _body_source_code; } -TileOperand &CLKernelWriter::declare_tile(const std::string &name, const TileInfo &tile_info) +TileOperand CLKernelWriter::declare_tile(const std::string &name, const TileInfo &tile_info) { const std::string fullname = generate_full_name(name); - const int32_t height = tile_info.height(); - const int32_t width = tile_info.width(); + const int32_t height = tile_info.height(); + const int32_t width = tile_info.width(); const DataType data_type = tile_info.data_type(); for(int32_t row = 0; row < height; ++row) @@ -75,13 +76,12 @@ TileOperand &CLKernelWriter::declare_tile(const std::string &name, const TileInf append_code(cl_type, " ", fullname, std::to_string(row), ";\n"); } - return add_operand(fullname, tile_info); -} + auto tile = std::make_unique<CLTile>(name, tile_info); + const auto operand = create_tile_operand(*tile); -TileOperand &CLKernelWriter::add_operand(const std::string &name, const TileInfo &tile_info) -{ - std::unique_ptr<TileOperand> operand = std::make_unique<CLTile>(name, tile_info); - return KernelWriter::add_operand(operand); + _tiles.insert(std::move(tile)); + + return operand; } } // namespace ckw diff --git a/compute_kernel_writer/src/cl/CLKernelWriter.h b/compute_kernel_writer/src/cl/CLKernelWriter.h index 5bf7293ae2..c69a0bc07e 100644 --- a/compute_kernel_writer/src/cl/CLKernelWriter.h +++ b/compute_kernel_writer/src/cl/CLKernelWriter.h @@ -26,7 +26,10 @@ #define CKW_SRC_CL_CLKERNELWRITER_H #include "ckw/KernelWriter.h" +#include "src/cl/CLTile.h" +#include <memory> +#include <set> #include <utility> namespace ckw @@ -62,8 +65,7 @@ public: * * Similar to @ref KernelWriter::declare_tile() */ - TileOperand &declare_tile(const ::std::string &name, const TileInfo &tile_info) override; - + TileOperand declare_tile(const std::string &name, const TileInfo &tile_info) override; protected: /** Append the specified code to the kernel body source code. */ @@ -84,9 +86,6 @@ protected: /** Get the current kernel body source code. */ const std::string &body_source_code() const; - /** Add a tile operand to the kernel and return it */ - TileOperand &add_operand(const std::string &code, const TileInfo &tile_info) override; - private: /** This string contains the kernel body source code, not the full CL source code. * The full source code will only be generated when the user calls @ref KernelWriter::emit_kernel. @@ -95,6 +94,8 @@ private: * Do not attempt to concatenate and alter this string directly. */ std::string _body_source_code{}; + + std::set<std::unique_ptr<CLTile>> _tiles{}; }; } // namespace ckw diff --git a/compute_kernel_writer/src/cl/CLTile.h b/compute_kernel_writer/src/cl/CLTile.h index b9d4bbaf84..f06bb449c0 100644 --- a/compute_kernel_writer/src/cl/CLTile.h +++ b/compute_kernel_writer/src/cl/CLTile.h @@ -24,10 +24,7 @@ #ifndef COMPUTE_KERNEL_WRITER_SRC_CL_CLTILE_H #define COMPUTE_KERNEL_WRITER_SRC_CL_CLTILE_H -#include "src/ITile.h" #include "src/cl/ICLTile.h" -#include "ckw/TileOperand.h" - #include <string> namespace ckw @@ -36,7 +33,7 @@ namespace ckw class TileInfo; /** OpenCL specific tile */ -class CLTile : public ICLTile, public TileOperand +class CLTile : public ICLTile { public: /** Constructor @@ -47,15 +44,15 @@ public: CLTile(const std::string &name, const TileInfo &info); // Inherited method overridden - TileVariable scalar(int32_t row, int32_t col) const override; + TileVariable scalar(int32_t row, int32_t col) const override; - TileVariable vector(int32_t row) const override; + TileVariable vector(int32_t row) const override; - TileVariable vector(int32_t row, int32_t col_start, int32_t width) const override; + TileVariable vector(int32_t row, int32_t col_start, int32_t width) const override; std::vector<TileVariable> all() const override; - bool is_assignable() const override; + bool is_assignable() const override; private: std::string create_var_name(int32_t row) const; |