From 67e3b0c63c35769c1ba28fa2a32446332bb4fcef Mon Sep 17 00:00:00 2001 From: Siddhesh Poyarekar Date: Wed, 6 Apr 2022 20:53:24 +0530 Subject: tests/string: Drop simple/stupid/builtin tests In most cases the simple/stupid/builtin functions were in there to benchmark optimized implementations against. Only in some cases the functions are used to check expected results. Remove these tests from IMPL() and only keep them in wherever they're used for a specific purpose, e.g. to generate expected results. This improves timing of `make subdirs=string` by over a minute and a half (over 15%) on a Whiskey Lake laptop. Signed-off-by: Siddhesh Poyarekar Reviewed-by: Noah Goldstein --- string/test-strlen.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'string/test-strlen.c') diff --git a/string/test-strlen.c b/string/test-strlen.c index dd394dadc5..21a9cc10a6 100644 --- a/string/test-strlen.c +++ b/string/test-strlen.c @@ -37,6 +37,7 @@ typedef size_t (*proto_t) (const CHAR *); +/* Naive implementation to verify results. */ size_t simple_STRLEN (const CHAR *s) { @@ -55,7 +56,6 @@ builtin_strlen (const CHAR *p) IMPL (builtin_strlen, 0) #endif -IMPL (simple_STRLEN, 0) IMPL (STRLEN, 1) -- cgit v1.2.3