diff options
author | Anthony Barbier <anthony.barbier@arm.com> | 2018-07-12 13:26:27 +0100 |
---|---|---|
committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:54:54 +0000 |
commit | e88b9bb3e2c97bc2c3f5024f17fa6c5080ee522c (patch) | |
tree | 44b5b4e7849bea992439152a944d1bc08c683a36 /tests/validate_examples/RunExample.cpp | |
parent | 7784c837afd5844fb6dc4d166ff253d983abfd2d (diff) | |
download | ComputeLibrary-e88b9bb3e2c97bc2c3f5024f17fa6c5080ee522c.tar.gz |
COMPMID-1396: make validate_example and benchmark_example check if setup was successful
Change-Id: I70dbd9f9258174fc80928906c33abecf804284c3
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/139716
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Tested-by: Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'tests/validate_examples/RunExample.cpp')
-rw-r--r-- | tests/validate_examples/RunExample.cpp | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/tests/validate_examples/RunExample.cpp b/tests/validate_examples/RunExample.cpp index b4e5d37648..d842c5bd6f 100644 --- a/tests/validate_examples/RunExample.cpp +++ b/tests/validate_examples/RunExample.cpp @@ -68,21 +68,30 @@ public: void do_setup() override { ARM_COMPUTE_ERROR_ON_NULLPTR(g_example.get()); - g_example->do_setup(g_example_argv.size(), &g_example_argv[0]); + _is_setup = g_example->do_setup(g_example_argv.size(), &g_example_argv[0]); } void do_run() override { - g_example->do_run(); + if(_is_setup) + { + g_example->do_run(); + } } void do_teardown() override { - if(validate) + if(_is_setup) { - g_example->do_validate(); + if(validate) + { + g_example->do_validate(); + } + g_example->do_teardown(); } - g_example->do_teardown(); g_example = nullptr; } + +private: + bool _is_setup{ false }; }; int run_example(int argc, char **argv, std::unique_ptr<ValidateExample> example) |