diff options
author | I-Jui (Ray) Sung <ijsung@google.com> | 2017-10-06 13:22:39 -0700 |
---|---|---|
committer | Ray Sung <ijsung@google.com> | 2017-10-09 20:57:42 +0000 |
commit | 5bf4edfb9fb7b1e0893609058dcfc8ff67be17cb (patch) | |
tree | ed11f34532e60b4ff071e395c724d713ea0710be | |
parent | 239a4504389c92528a7eabff4d220a97e56d2121 (diff) | |
download | android_hardware_interfaces-5bf4edfb9fb7b1e0893609058dcfc8ff67be17cb.tar.gz android_hardware_interfaces-5bf4edfb9fb7b1e0893609058dcfc8ff67be17cb.tar.bz2 android_hardware_interfaces-5bf4edfb9fb7b1e0893609058dcfc8ff67be17cb.zip |
Refactor test harness for generated VTS tests
Address comments in ag/2926388
Test: VtsHalNeuralnetworksV1_0TargetTest on Angler with sample driver
Bug: 67597726
Change-Id: Ie3118dcbfb11c25dbc84d2becdefbbae868ff28f
-rw-r--r-- | neuralnetworks/1.0/vts/functional/GeneratedTestHarness.cpp | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/neuralnetworks/1.0/vts/functional/GeneratedTestHarness.cpp b/neuralnetworks/1.0/vts/functional/GeneratedTestHarness.cpp index 735528b95..d740b5f53 100644 --- a/neuralnetworks/1.0/vts/functional/GeneratedTestHarness.cpp +++ b/neuralnetworks/1.0/vts/functional/GeneratedTestHarness.cpp @@ -46,11 +46,11 @@ using ::generated_tests::Int32Operands; using ::generated_tests::Quant8Operands; using ::generated_tests::compare; -template <typename ty> +template <typename T> void copy_back_(MixedTyped* dst, const std::vector<RequestArgument>& ra, char* src) { MixedTyped& test = *dst; - for_each(test, [&ra, src](int index, std::vector<ty>& m) { - ASSERT_EQ(m.size(), ra[index].location.length / sizeof(ty)); + for_each<T>(test, [&ra, src](int index, std::vector<T>& m) { + ASSERT_EQ(m.size(), ra[index].location.length / sizeof(T)); char* begin = src + ra[index].location.offset; memcpy(m.data(), begin, ra[index].location.length); }); @@ -213,10 +213,8 @@ void Execute(const sp<IDevice>& device, std::function<Model(void)> create_model, copy_back(&test, outputs_info, outputPtr); outputMemory->commit(); // Filter out don't cares - MixedTyped filtered_golden; - MixedTyped filtered_test; - filter(golden, &filtered_golden, is_ignored); - filter(test, &filtered_test, is_ignored); + MixedTyped filtered_golden = filter(golden, is_ignored); + MixedTyped filtered_test = filter(test, is_ignored); // We want "close-enough" results for float compare(filtered_golden, filtered_test); |