aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlexandre Rames <alexandre.rames@linaro.org>2016-09-07 09:35:04 +0100
committerAlexandre Rames <alexandre.rames@linaro.org>2016-09-07 09:35:04 +0100
commitc6837357d45ec9e795f9138a003103c1994b2871 (patch)
tree8005fdab9077477d3e8ab9b7ee4cad22f3f53aa6
parent8b4ac36f9143477148a9ea83c3e00998fc2c02fd (diff)
downloadart-testing-c6837357d45ec9e795f9138a003103c1994b2871.tar.gz
Update instructions for verify methods in README.md
Change-Id: Iebb3359fa7efe9bbd60fac5b4471e1a73d520c00
-rw-r--r--README.md14
1 files changed, 10 insertions, 4 deletions
diff --git a/README.md b/README.md
index 3e9d043..4b74ae4 100644
--- a/README.md
+++ b/README.md
@@ -162,6 +162,8 @@ existing benchmark. Besides, developers should also notice:
methods and run them.
2. Verify methods start with "verify" -- all boolean verifyXXX() methods will
be run to check the benchmark is working correctly.
+ `verify` methods should *not* depend on the benchmark having run before it is
+ called.
3. Leave iterations as parameter -- Test launcher will fill it with a value
to make sure it runs in a reasonable duration.
4. Without auto-calibration benchmarks should run for a reasonable amount of
@@ -190,14 +192,18 @@ existing benchmark. Besides, developers should also notice:
int result = 0;
for (int i = 0; i < iters; i++) {
// test code
- testAddResults[i] = i + i;
+ result += i;
+ testAddResults[i] = result;
}
return result;
}
- public boolean verifyTestAdd() {
- boolean result = // test contents of testAddResults[]
- return result;
+ public static boolean verifyTestAdd() {
+ return timeTestAdd(0) == 0 &&
+ timeTestAdd(1) == 1 &&
+ timeTestAdd(2) == 3 &&
+ timeTestAdd(100) == 5050 &&
+ timeTestAdd(123) == 7626;
}
// If you want to fill iterations with your own value. Write a method like: