diff options
author | Georgios Pinitas <georgios.pinitas@arm.com> | 2018-08-30 16:02:11 +0100 |
---|---|---|
committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:54:54 +0000 |
commit | 09d3451b4caf8d5e0e7cf2c6097e50a1f815d027 (patch) | |
tree | 8e9b8e9882d0120c85ff08473dbf3721b7602677 /tests/benchmark/NEON | |
parent | b57be0da77370e5e71fe82dfa281f528279e8127 (diff) | |
download | ComputeLibrary-09d3451b4caf8d5e0e7cf2c6097e50a1f815d027.tar.gz |
COMPMID-1548: NEON FP16 mismatches on CannyEdge and HarrisCorners.
Removes FP16 from HarrisCorners and CannyEdge.
Change-Id: I5e4f9205fdbe4de85f04f55ecf1568c837e56cc0
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/146247
Tested-by: Jenkins <bsgcomp@arm.com>
Reviewed-by: Michele DiGiorgio <michele.digiorgio@arm.com>
Diffstat (limited to 'tests/benchmark/NEON')
-rw-r--r-- | tests/benchmark/NEON/CannyEdge.cpp | 3 | ||||
-rw-r--r-- | tests/benchmark/NEON/HarrisCorners.cpp | 34 |
2 files changed, 5 insertions, 32 deletions
diff --git a/tests/benchmark/NEON/CannyEdge.cpp b/tests/benchmark/NEON/CannyEdge.cpp index 50fd832e5d..c4414507c9 100644 --- a/tests/benchmark/NEON/CannyEdge.cpp +++ b/tests/benchmark/NEON/CannyEdge.cpp @@ -42,10 +42,9 @@ namespace { // *INDENT-OFF* // clang-format off -const auto use_fp16 = framework::dataset::make("UseFP16", { false }); const auto canny_edge_dataset = combine(framework::dataset::make("GradientSize", { 3, 5, 7 }), combine(framework::dataset::make("Normalization", { MagnitudeType::L1NORM, MagnitudeType::L2NORM }), - combine(datasets::BorderModes(), use_fp16))); + datasets::BorderModes())); } // namespace using NECannyEdgeFixture = CannyEdgeFixture<Tensor, NECannyEdge, Accessor>; diff --git a/tests/benchmark/NEON/HarrisCorners.cpp b/tests/benchmark/NEON/HarrisCorners.cpp index 33315ffcd0..75a6794cc5 100644 --- a/tests/benchmark/NEON/HarrisCorners.cpp +++ b/tests/benchmark/NEON/HarrisCorners.cpp @@ -53,49 +53,23 @@ using NEHarrisCornersFixture = HarrisCornersFixture<Tensor, NEHarrisCorners, Acc TEST_SUITE(NEON) TEST_SUITE(HarrisCorners) -#ifdef __ARM_FEATURE_FP16_VECTOR_ARITHMETIC -TEST_SUITE(FP16) - -REGISTER_FIXTURE_DATA_TEST_CASE(RunSmall, NEHarrisCornersFixture, framework::DatasetMode::PRECOMMIT, combine(combine(combine(combine(combine(combine(combine(combine(datasets::SmallImageFiles(), - framework::dataset::make("Format", { Format::U8 })), - threshold), - min_dist), - sensitivity), - gradient_size), - block_size), - border_mode), - framework::dataset::make("UseFP16", { true }))); -REGISTER_FIXTURE_DATA_TEST_CASE(RunLarge, NEHarrisCornersFixture, framework::DatasetMode::NIGHTLY, combine(combine(combine(combine(combine(combine(combine(combine(datasets::LargeImageFiles(), - framework::dataset::make("Format", { Format::U8 })), - threshold), - min_dist), - sensitivity), - gradient_size), - block_size), - border_mode), - framework::dataset::make("UseFP16", { true }))); -TEST_SUITE_END() // FP16 -#endif // __ARM_FEATURE_FP16_VECTOR_ARITHMETIC - TEST_SUITE(S16) -REGISTER_FIXTURE_DATA_TEST_CASE(RunSmall, NEHarrisCornersFixture, framework::DatasetMode::PRECOMMIT, combine(combine(combine(combine(combine(combine(combine(combine(datasets::SmallImageFiles(), +REGISTER_FIXTURE_DATA_TEST_CASE(RunSmall, NEHarrisCornersFixture, framework::DatasetMode::PRECOMMIT, combine(combine(combine(combine(combine(combine(combine(datasets::SmallImageFiles(), framework::dataset::make("Format", { Format::U8 })), threshold), min_dist), sensitivity), gradient_size), block_size), - border_mode), - framework::dataset::make("UseFP16", { false }))); -REGISTER_FIXTURE_DATA_TEST_CASE(RunLarge, NEHarrisCornersFixture, framework::DatasetMode::NIGHTLY, combine(combine(combine(combine(combine(combine(combine(combine(datasets::LargeImageFiles(), + border_mode)); +REGISTER_FIXTURE_DATA_TEST_CASE(RunLarge, NEHarrisCornersFixture, framework::DatasetMode::NIGHTLY, combine(combine(combine(combine(combine(combine(combine(datasets::LargeImageFiles(), framework::dataset::make("Format", { Format::U8 })), threshold), min_dist), sensitivity), gradient_size), block_size), - border_mode), - framework::dataset::make("UseFP16", { false }))); + border_mode)); TEST_SUITE_END() // S16 TEST_SUITE_END() // HarrisCorners TEST_SUITE_END() // NEON |