tests: flatten nested combine usage in NEON validation

[morgolock]

Change-Id: I976e01b9be9ee91f7de09f050392576133d1e929

[gunes-arm]

I think the commit type should be test, but I guess I have a better suggestion. Isn't this a refactor?

[gunes-arm]

Can you also count the number of tests before & after this patch as a sanity check, and report?

[morgolock]

I think the commit type should be test, but I guess I have a better suggestion. Isn't this a refactor?

I changed the commit to

refactor: flatten nested combine usage in NEON validation

[morgolock]

Can you also count the number of tests before & after this patch as a sanity check, and report?

With this patch --mode=precommit

Executed 78213 test(s) (78213 passed, 0 expected failures, 0 failed, 0 crashed, 0 disabled) in 247 second(s)

Without this patch

Executed 78213 test(s) (78213 passed, 0 expected failures, 0 failed, 0 crashed, 0 disabled) in 258 second(s)

[gunes-arm]

Thanks for checking the number of tests. Can you check for --mode=ALL? You only need to list the tests, not run, i.e.

./arm_compute_validation --mode=ALL --list-tests | wc -l

[morgolock]

Thanks for checking the number of tests. Can you check for --mode=ALL? You only need to list the tests, not run, i.e.

./arm_compute_validation --mode=ALL --list-tests | wc -l

Both cases I get

130|shiba:/data/local/tmp/pabtel01 $ ./arm_compute_validation  --mode=ALL --list-tests | wc -l                                                                                        
425731

[gunes-arm]

Copyright years did not change