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>
diff --git a/compute_kernel_writer/validation/tests/CLConstantTileTest.hpp b/compute_kernel_writer/validation/tests/CLConstantTileTest.hpp
index 23a75c4..f10ad10 100644
--- a/compute_kernel_writer/validation/tests/CLConstantTileTest.hpp
+++ b/compute_kernel_writer/validation/tests/CLConstantTileTest.hpp
@@ -22,13 +22,13 @@
* SOFTWARE.
*/
-#ifndef COMPUTE_KERNEL_WRITER_TESTS_CLCONSTANTTILETEST_HPP
-#define COMPUTE_KERNEL_WRITER_TESTS_CLCONSTANTTILETEST_HPP
+#ifndef CKW_TESTS_CLCONSTANTTILETEST_HPP
+#define CKW_TESTS_CLCONSTANTTILETEST_HPP
#include "common/Common.h"
#include "src/Helpers.h"
-#include "src/cl/CLConstantTile.h"
#include "src/cl/CLHelpers.h"
+#include "src/cl/CLTile.h"
#include <random>
#include <string>
@@ -55,10 +55,10 @@
int32_t test_idx = 0;
for(const auto &test : _values)
{
- const CLConstantTile tile(test, DataType::Fp16);
- const auto vars = tile.all();
- const int32_t num_vars = vars.size();
- const int32_t width = tile.info().width();
+ const CLTile tile(test, DataType::Fp16);
+ const auto vars = tile.all();
+ const int32_t num_vars = vars.size();
+ const int32_t width = tile.info().width();
for(int32_t y = 0; y < num_vars; ++y)
{
@@ -136,7 +136,7 @@
}
}
- const CLConstantTile tile(container, dt);
+ const CLTile tile(container, dt);
const TileVariable var = tile.scalar(y_coord, x_coord);
@@ -214,7 +214,7 @@
}
}
- const CLConstantTile tile(container, dt);
+ const CLTile tile(container, dt);
const TileVariable var = tile.scalar(y_coord, x_coord);
@@ -260,9 +260,9 @@
for(const auto &test : _values)
{
- const CLConstantTile tile(test, dt);
- const int32_t width = tile.info().width();
- const int32_t height = tile.info().height();
+ const CLTile tile(test, dt);
+ const int32_t width = tile.info().width();
+ const int32_t height = tile.info().height();
for(int32_t row = 0; row < height; ++row)
{
@@ -326,8 +326,8 @@
{
for(auto &subwidth : _subwidths)
{
- const CLConstantTile tile(test, dt);
- const int32_t height = tile.info().height();
+ const CLTile tile(test, dt);
+ const int32_t height = tile.info().height();
for(int32_t row = 0; row < height; ++row)
{
@@ -368,4 +368,4 @@
} // namespace ckw
-#endif /* COMPUTE_KERNEL_WRITER_TESTS_CLCONSTANTTILETEST_HPP */
+#endif // CKW_TESTS_CLCONSTANTTILETEST_HPP