diff options
author | Viet-Hoa Do <viet-hoa.do@arm.com> | 2023-06-29 13:56:17 +0100 |
---|---|---|
committer | Viet-Hoa Do <viet-hoa.do@arm.com> | 2023-06-29 16:06:35 +0000 |
commit | 7644b912128e64078e4bf852ef5b17111e5e4012 (patch) | |
tree | 5e10900c353f7bd2fc95dce89d92e53cd5696c14 /compute_kernel_writer/validation | |
parent | 00474e99260da69c5abd14277d0dd0b6de209904 (diff) | |
download | ComputeLibrary-7644b912128e64078e4bf852ef5b17111e5e4012.tar.gz |
Fix CKW test failure
* TileInfo constructor no longer takes width and height. It now
takes height then width.
Partially resolves: COMPMID-6283
Signed-off-by: Viet-Hoa Do <viet-hoa.do@arm.com>
Change-Id: I6284de98ecaf53c1013a90bf0b71f60bb36b7c48
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/9844
Reviewed-by: Jakub Sujak <jakub.sujak@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/validation')
-rw-r--r-- | compute_kernel_writer/validation/Validation.cpp | 2 | ||||
-rw-r--r-- | compute_kernel_writer/validation/tests/CLTileTest.hpp | 14 |
2 files changed, 8 insertions, 8 deletions
diff --git a/compute_kernel_writer/validation/Validation.cpp b/compute_kernel_writer/validation/Validation.cpp index da0ce8ad15..0a288837d8 100644 --- a/compute_kernel_writer/validation/Validation.cpp +++ b/compute_kernel_writer/validation/Validation.cpp @@ -88,7 +88,7 @@ int32_t main() } else { - std::runtime_error("One or more tests failed"); + throw std::runtime_error("One or more tests failed"); } return 0; diff --git a/compute_kernel_writer/validation/tests/CLTileTest.hpp b/compute_kernel_writer/validation/tests/CLTileTest.hpp index f3fdcd1304..7847f0e128 100644 --- a/compute_kernel_writer/validation/tests/CLTileTest.hpp +++ b/compute_kernel_writer/validation/tests/CLTileTest.hpp @@ -53,7 +53,7 @@ public: // The status of this variable can change in VALIDATE_TEST() bool all_tests_passed = true; - const TileInfo info(dt, width, height); + const TileInfo info(dt, height, width); int32_t test_idx = 0; for(const auto &tile_name : _tile_name) @@ -108,7 +108,7 @@ public: { const int32_t width = _width[i]; const int32_t height = _height[i]; - const TileInfo info(DataType::Fp32, width, height); + const TileInfo info(DataType::Fp32, height, width); const CLTile tile("src", info); const auto vars = tile.all(); const int32_t num_vars = vars.size(); @@ -152,7 +152,7 @@ public: bool run() override { - const TileInfo info(dt, width, height); + const TileInfo info(dt, height, width); const CLTile tile(tile_name, info); VALIDATE_ON_MSG(_x_coord.size() == _y_coord.size(), "The number of x-coords and y-coords does not match"); @@ -231,7 +231,7 @@ public: const int32_t x_coord_clamped = clamp(x_coord, static_cast<int32_t>(0), width - 1); - const TileInfo info(dt, width, height); + const TileInfo info(dt, height, width); const CLTile tile(tile_name, info); const TileVariable var = tile.scalar(y_coord, x_coord); @@ -301,7 +301,7 @@ public: const int32_t y_coord_clamped = clamp(y_coord, static_cast<int32_t>(0), height - 1); - const TileInfo info(dt, width, height); + const TileInfo info(dt, height, width); const CLTile tile(tile_name, info); const TileVariable var = tile.scalar(y_coord, x_coord); @@ -356,7 +356,7 @@ public: int32_t test_idx = 0; for(const auto &height : _heights) { - const TileInfo info(dt, width, height); + const TileInfo info(dt, height, width); const CLTile tile(tile_name, info); for(int32_t row = 0; row < height; ++row) @@ -415,7 +415,7 @@ public: { for(const auto &subwidth : _subwidths) { - const TileInfo info(dt, width, height); + const TileInfo info(dt, height, width); const CLTile tile(tile_name, info); for(int32_t row = 0; row < height; ++row) |