diff options
author | Siddhesh Poyarekar <siddhesh@redhat.com> | 2013-12-05 10:12:59 +0530 |
---|---|---|
committer | Siddhesh Poyarekar <siddhesh@redhat.com> | 2013-12-05 10:12:59 +0530 |
commit | 9298ecba15e2b8055e68189c1b11b08ef3ac008d (patch) | |
tree | 1ba9f9dcb8c625e851da8d978778d06a74b8f47a /benchtests | |
parent | 232983e9a74e817377a5e76f2c3872c8a92685d0 (diff) | |
download | glibc-9298ecba15e2b8055e68189c1b11b08ef3ac008d.tar.gz |
Accept output arguments to benchmark functions
This patch adds the ability to accept output arguments to functions
being benchmarked, by nesting the argument type in <> in the args
directive. It includes the sincos implementation as an example, where
the function would have the following args directive:
## args: double:<double *>:<double *>
This simply adds a definition for a static variable whose pointer gets
passed into the function, so it's not yet possible to pass something
more complicated like a pre-allocated string or array. That would be
a good feature to add if a function needs it.
The values in the input file will map only to the input arguments. So
if I had a directive like this for a function foo:
## args: int:<int *>:int:<int *>
and I have a value list like this:
1, 2
3, 4
5, 6
then the function calls generated would be:
foo (1, &out1, 2, &out2);
foo (3, &out1, 4, &out2);
foo (5, &out1, 6, &out2);
Diffstat (limited to 'benchtests')
-rw-r--r-- | benchtests/README | 3 | ||||
-rw-r--r-- | benchtests/bench-sincos.c | 86 | ||||
-rw-r--r-- | benchtests/sincos-inputs | 27 |
3 files changed, 29 insertions, 87 deletions
diff --git a/benchtests/README b/benchtests/README index 0c643a6590..a5fd8dafe3 100644 --- a/benchtests/README +++ b/benchtests/README @@ -48,7 +48,8 @@ one to add `foo' to the bench tests: - args: This should be assigned a colon separated list of types of the input arguments. This directive may be skipped if the function does not take any - inputs. + inputs. One may identify output arguments by nesting them in <>. The + generator will create variables to get outputs from the calling function. - ret: This should be assigned the type that the function returns. This directive may be skipped if the function does not return a value. - includes: This should be assigned a comma-separated list of headers that diff --git a/benchtests/bench-sincos.c b/benchtests/bench-sincos.c deleted file mode 100644 index 7662dde04a..0000000000 --- a/benchtests/bench-sincos.c +++ /dev/null @@ -1,86 +0,0 @@ -/* Copyright (C) 2013 Free Software Foundation, Inc. - This file is part of the GNU C Library. - - The GNU C Library is free software; you can redistribute it and/or - modify it under the terms of the GNU Lesser General Public - License as published by the Free Software Foundation; either - version 2.1 of the License, or (at your option) any later version. - - The GNU C Library is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - Lesser General Public License for more details. - - You should have received a copy of the GNU Lesser General Public - License along with the GNU C Library; if not, see - <http://www.gnu.org/licenses/>. */ - -extern void sincos (double, double *, double *); - -#define CALL_BENCH_FUNC(v, i, j, k) sincos ( variants[v].in[i].arg0, &j, &k); - -struct args -{ - volatile double arg0; -}; - -struct args in0[12] = -{ - { 0.9 }, - { 2.3 }, - { 3.7 }, - { 3.9 }, - { 4.0 }, - { 4.7 }, - { 5.9 }, - - { 0x1.000000cf4a2a1p0 }, - { 0x1.0000010b239a8p0 }, - { 0x1.00000162a932ap0 }, - { 0x1.000002d452a11p0 }, - { 0x1.000005bc7d86cp0 } -}; - -struct args in1[12] = -{ - { 0.93340582292648832662962377071381 }, - { 2.3328432680770916363144351635128 }, - { 3.7439477503636453548097051680088 }, - { 3.9225160069792437411706487182528 }, - { 4.0711651639931289992091478779912 }, - { 4.7858438478542097982426639646292 }, - { 5.9840767662578002727968851104379 }, - - { 0x1.000000cf4a2a2p0 }, - { 0x1.0000010b239a9p0 }, - { 0x1.00000162a932bp0 }, - { 0x1.000002d452a10p0 }, - { 0x1.000005bc7d86dp0 } -}; - -struct _variants -{ - const char *name; - int count; - struct args *in; -}; - -struct _variants variants[2] = - { - {"sincos()", 12, in0}, - {"sincos(768bits)", 12, in1}, - }; - -#define NUM_VARIANTS 2 -#define NUM_SAMPLES(i) (variants[i].count) -#define VARIANT(i) (variants[i].name) - -#define BENCH_FUNC(v, j) \ -({ \ - volatile double iptr; \ - volatile double iptr2; \ - CALL_BENCH_FUNC (v, j, iptr, iptr2); \ -}) - -#define FUNCNAME "sincos" -#include "bench-skeleton.c" diff --git a/benchtests/sincos-inputs b/benchtests/sincos-inputs new file mode 100644 index 0000000000..3df4d2a23b --- /dev/null +++ b/benchtests/sincos-inputs @@ -0,0 +1,27 @@ +## includes: math.h +## args: double:<double *>:<double *> +0.9 +2.3 +3.7 +3.9 +4.0 +4.7 +5.9 +0x1.000000cf4a2a1p0 +0x1.0000010b239a8p0 +0x1.00000162a932ap0 +0x1.000002d452a11p0 +0x1.000005bc7d86cp0 +## name: 768bits +0.93340582292648832662962377071381 +2.3328432680770916363144351635128 +3.7439477503636453548097051680088 +3.9225160069792437411706487182528 +4.0711651639931289992091478779912 +4.7858438478542097982426639646292 +5.9840767662578002727968851104379 +0x1.000000cf4a2a2p0 +0x1.0000010b239a9p0 +0x1.00000162a932bp0 +0x1.000002d452a10p0 +0x1.000005bc7d86dp0 |