diff options
Diffstat (limited to 'compute_kernel_writer/src/cl')
-rw-r--r-- | compute_kernel_writer/src/cl/CLKernelWriter.cpp | 25 | ||||
-rw-r--r-- | compute_kernel_writer/src/cl/CLKernelWriter.h | 15 |
2 files changed, 35 insertions, 5 deletions
diff --git a/compute_kernel_writer/src/cl/CLKernelWriter.cpp b/compute_kernel_writer/src/cl/CLKernelWriter.cpp index 312162f498..79d0f985d0 100644 --- a/compute_kernel_writer/src/cl/CLKernelWriter.cpp +++ b/compute_kernel_writer/src/cl/CLKernelWriter.cpp @@ -28,6 +28,7 @@ #include "ckw/Kernel.h" #include "ckw/TensorSampler.h" #include "ckw/TileOperand.h" +#include "ckw/types/DataType.h" #include "ckw/types/MemoryOperation.h" #include "ckw/types/TargetLanguage.h" #include "src/ITensorComponent.h" @@ -37,7 +38,6 @@ #include "src/cl/helpers/CLMemoryOpBufferHelper.h" #include "src/cl/helpers/CLMemoryOpImage2dHelper.h" #include "src/cl/helpers/ICLMemoryOpHelper.h" - #include "src/types/DataTypeHelpers.h" #include <algorithm> @@ -364,7 +364,7 @@ TileOperand CLKernelWriter::declare_tile(const std::string &name, const TileInfo return e->name() == fullname; }) == _tiles.end(), - "Tile name must be unique."); + "There is already a tile with name: " + fullname); auto tile = std::make_unique<CLTile>(fullname, tile_info); @@ -381,17 +381,27 @@ TileOperand CLKernelWriter::declare_tile(const std::string &name, const TileInfo return operand; } +TileOperand CLKernelWriter::declare_constant_tile(const ConstantData &data) +{ + auto tile = std::make_unique<CLTile>(get_values(data), get_data_type(data)); + const TileOperand operand = create_tile_operand(*tile); + _constant_tiles.insert(std::move(tile)); + + return operand; +} + void CLKernelWriter::op_write_raw_code(const std::string &raw_code) { append_code(raw_code); } -const CLTile &CLKernelWriter::to_cl_tile(const TileOperand &operand) +const CLTile &CLKernelWriter::to_cl_tile(const TileOperand &operand) const { const auto &tile = get_tile(operand); #ifdef COMPUTE_KERNEL_WRITER_ASSERTS_ENABLED // Check if the tile is a CLTile created by this kernel writer. + { bool found = false; @@ -404,6 +414,15 @@ const CLTile &CLKernelWriter::to_cl_tile(const TileOperand &operand) } } + for(const auto &t : _constant_tiles) + { + if(&tile == t.get()) + { + found = true; + break; + } + } + if(!found) { for(const auto &t : _tensors) diff --git a/compute_kernel_writer/src/cl/CLKernelWriter.h b/compute_kernel_writer/src/cl/CLKernelWriter.h index 2a6b79c691..d2c84f192e 100644 --- a/compute_kernel_writer/src/cl/CLKernelWriter.h +++ b/compute_kernel_writer/src/cl/CLKernelWriter.h @@ -29,17 +29,21 @@ #include <memory> #include <set> +#include <string> #include <utility> namespace ckw { +// Forward Declarations class CLTile; class CLTensorArgument; +class ConstantData; +class TensorOperand; class TensorSampler; class TileOperand; -class TensorOperand; +enum class DataType; enum class MemoryOperation; /** OpenCL kernel writer. */ @@ -96,6 +100,12 @@ public: */ TileOperand declare_tile(const std::string &name, const TileInfo &tile_info) override; + /** Declare a constant tile given a @ref:ConstantData object + * + * Similar to @ref KernelWriter::declare_constant_tile() + */ + TileOperand declare_constant_tile(const ConstantData &data) override; + // ============================================================================================= // Memory Operations // ============================================================================================= @@ -139,7 +149,7 @@ protected: * * This function performs appropriate check before doing type casting. */ - const CLTile &to_cl_tile(const TileOperand &operand); + const CLTile &to_cl_tile(const TileOperand &operand) const; /** Append the specified code to the kernel body source code. */ template <typename T, typename... TArgs> @@ -179,6 +189,7 @@ private: std::set<std::unique_ptr<CLTensorArgument>> _tensors{}; std::set<std::unique_ptr<CLTile>> _tiles{}; + std::set<std::unique_ptr<CLTile>> _constant_tiles{}; }; } // namespace ckw |