diff options
author | Viet-Hoa Do <viet-hoa.do@arm.com> | 2023-09-19 16:41:34 +0100 |
---|---|---|
committer | Viet-Hoa Do <viet-hoa.do@arm.com> | 2023-09-22 12:07:09 +0000 |
commit | cd1f03e765ad0f3ca3b68b1a7c1d0a1539cab439 (patch) | |
tree | 9cb78579e01e14501c316f5297c804ba13c8ad37 /compute_kernel_writer/src/cl/CLTile.cpp | |
parent | 1f841a52f9a7f52948d676bc3807461bbed6f70a (diff) | |
download | ComputeLibrary-cd1f03e765ad0f3ca3b68b1a7c1d0a1539cab439.tar.gz |
Add row vector and scalar access support to tile operand
* Add the concept of tile view which refers to a specific rectangular
area of the tile object.
- The active area is added to TileOperand so that the user can access
part of the tile.
- Currently only row vector and scalar access are exposed to the user.
- All writing operations except load/store op support sub-tile.
* Add tests for sub-tile access.
Resolves: COMPMID-6557
Signed-off-by: Mohammed Suhail Munshi <MohammedSuhail.Munshi@arm.com>
Signed-off-by: Viet-Hoa Do <viet-hoa.do@arm.com>
Change-Id: Ica3f9eaf17f06e080c495d36c572f623b62c2910
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/10354
Reviewed-by: Gunes Bayir <gunes.bayir@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Benchmark: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'compute_kernel_writer/src/cl/CLTile.cpp')
-rw-r--r-- | compute_kernel_writer/src/cl/CLTile.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/compute_kernel_writer/src/cl/CLTile.cpp b/compute_kernel_writer/src/cl/CLTile.cpp index 556db0f47b..0cce69a9e1 100644 --- a/compute_kernel_writer/src/cl/CLTile.cpp +++ b/compute_kernel_writer/src/cl/CLTile.cpp @@ -125,6 +125,9 @@ TileVariable CLTile::vector(int32_t row) const TileVariable CLTile::vector(int32_t row, int32_t col_start, int32_t width) const { + CKW_ASSERT(col_start >= 0 && col_start < _info.width()); + CKW_ASSERT(col_start + width <= _info.width()); + // Validate the new vector length cl_validate_vector_length(width); @@ -154,7 +157,7 @@ TileVariable CLTile::vector(int32_t row, int32_t col_start, int32_t width) const { t.str = create_var_name(row); - if(_info.width() != 1) + if(_info.width() != 1 && _info.width() != width) { t.str += ".s"; for(int i = 0; i < width; ++i) |