scottk/OpenCellular
1
0
Fork 0
mirror of https://github.com/Telecominfraproject/OpenCellular.git synced 2026-01-09 17:11:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: remove 64-bit multiplication

Browse Source 
On some CPU cores, the compiler needs an helper to perform the 64-bit
multiplication.
As the only 64-bit multiplication in the code base is not necessary, fix
it rather than adding the helper.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BRANCH=none
BUG=none
TEST=build for Cortex-M0 platform.

Change-Id: Id5d6b4b6641f81732a456dacb78dee7262f6729d
Reviewed-on: https://chromium-review.googlesource.com/188980
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Commit-Queue: Vincent Palatin <vpalatin@chromium.org>
Tested-by: Vincent Palatin <vpalatin@chromium.org>
This commit is contained in:
Vincent Palatin
2014-03-05 11:49:05 -08:00
committed by chrome-internal-fetch
parent f4f934c103
commit 75f59a47ec
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
test/utils.c
View File

@@ -99,7 +99,7 @@ static int test_memmove(void)
/* Expected about 4x speed gain. Use 3x because it fluctuates */
#ifndef TEST_COVERAGE
/* Measuring coverage makes it fluctuate even more, so skip it. */
TEST_ASSERT((t1.val-t0.val) > (t3.val-t2.val) * 3);
TEST_ASSERT((t1.val-t0.val) > (unsigned)(t3.val-t2.val) * 3);
#endif
/* Test small moves */
@@ -145,7 +145,7 @@ static int test_memcpy(void)
TEST_ASSERT_ARRAY_EQ(buf + dest_offset, buf, len);
/* Expected about 4x speed gain. Use 3x because it fluctuates */
TEST_ASSERT((t1.val-t0.val) > (t3.val-t2.val) * 3);
TEST_ASSERT((t1.val-t0.val) > (unsigned)(t3.val-t2.val) * 3);
memcpy(buf + dest_offset + 1, buf + 1, len - 1);
TEST_ASSERT_ARRAY_EQ(buf + dest_offset + 1, buf + 1, len - 1);