diff options
author | Viet-Hoa Do <viet-hoa.do@arm.com> | 2023-07-24 15:47:34 +0100 |
---|---|---|
committer | Viet-Hoa Do <viet-hoa.do@arm.com> | 2023-07-25 10:08:05 +0000 |
commit | 0250fa6c2a0bdbf88c1264f32ad0a1a4e3fec3f3 (patch) | |
tree | f5772b7df56c30f8ce8bd7fc182ba803c44b1691 /compute_kernel_writer/src/cl/CLTile.h | |
parent | 25d26f4d86042e0ca52ac1bef4039b187f77b5b3 (diff) | |
download | ComputeLibrary-0250fa6c2a0bdbf88c1264f32ad0a1a4e3fec3f3.tar.gz |
Use CLTile for both variable and constant tiles
* It's easier to reuse CLTile for other things for example
tensor component if it can represent both variable
and constant tiles.
Partially resolves: COMPMID-6391
Signed-off-by: Viet-Hoa Do <viet-hoa.do@arm.com>
Change-Id: Ief06f670332cb339bd31b94a31b4bec186e1f1b8
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/9966
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Benchmark: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'compute_kernel_writer/src/cl/CLTile.h')
-rw-r--r-- | compute_kernel_writer/src/cl/CLTile.h | 29 |
1 files changed, 26 insertions, 3 deletions
diff --git a/compute_kernel_writer/src/cl/CLTile.h b/compute_kernel_writer/src/cl/CLTile.h index f06bb449c0..46af4de364 100644 --- a/compute_kernel_writer/src/cl/CLTile.h +++ b/compute_kernel_writer/src/cl/CLTile.h @@ -24,7 +24,7 @@ #ifndef COMPUTE_KERNEL_WRITER_SRC_CL_CLTILE_H #define COMPUTE_KERNEL_WRITER_SRC_CL_CLTILE_H -#include "src/cl/ICLTile.h" +#include "src/ITile.h" #include <string> namespace ckw @@ -33,17 +33,31 @@ namespace ckw class TileInfo; /** OpenCL specific tile */ -class CLTile : public ICLTile +class CLTile : public ITile, public IVectorAccess, public IScalarAccess { public: - /** Constructor + /** Initialize a new instance of @ref CLTile class for variable tile. * * @param[in] name Tile name * @param[in] info Tile info */ CLTile(const std::string &name, const TileInfo &info); + /** Initialize a new instane of @ref CLTile class for compile-time constant tile. + * + * @note A constant tile does not need a name since this object does not return variable's name but rather + * values stored as string type + * + * @param[in] vals The tile container with the constant values as std::string + * @param[in] dt Datatype of the values stored in the tile container + */ + CLTile(const TileContainer &vals, DataType dt); + // Inherited method overridden + const std::string &name() const override; + + const TileInfo &info() const override; + TileVariable scalar(int32_t row, int32_t col) const override; TileVariable vector(int32_t row) const override; @@ -54,8 +68,17 @@ public: bool is_assignable() const override; + std::vector<int32_t> supported_vector_lengths() const override; + private: + void validate_tile_info(const TileInfo &info) const; + std::string create_var_name(int32_t row) const; + + TileInfo _info{ DataType::Unknown }; + std::string _basename{ "" }; + bool _is_constant{ false }; + TileContainer _vals{}; }; } // namespace ckw |