summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorI-Jui (Ray) Sung <ijsung@google.com>2017-10-06 13:22:39 -0700
committerRay Sung <ijsung@google.com>2017-10-09 20:57:42 +0000
commit5bf4edfb9fb7b1e0893609058dcfc8ff67be17cb (patch)
treeed11f34532e60b4ff071e395c724d713ea0710be
parent239a4504389c92528a7eabff4d220a97e56d2121 (diff)
downloadandroid_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.cpp12
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);