diff mbox

benchtests: Build ffs and ffsl benchtests with -fno-builtin

Message ID 1396347841-32675-1-git-send-email-will.newton@linaro.org
State Accepted
Headers show

Commit Message

Will Newton April 1, 2014, 10:24 a.m. UTC 
Without this flag it is possible that the compiler will optimize
away the calls to ffs/ffsll.

ChangeLog:

2014-03-31  Will Newton  <will.newton@linaro.org>

	* benchtests/Makefile (CFLAGS-bench-ffs.c): Add
	-fno-builtin.  (CFLAGS-bench-ffsll.c): Likewise.
---
 benchtests/Makefile | 3 +++
 1 file changed, 3 insertions(+)

Comments

Siddhesh Poyarekar April 1, 2014, 10:26 a.m. UTC | #1 
On 1 April 2014 15:54, Will Newton <will.newton@linaro.org> wrote:
> Without this flag it is possible that the compiler will optimize
> away the calls to ffs/ffsll.

Looks good to me.

Siddhesh

>
> ChangeLog:
>
> 2014-03-31  Will Newton  <will.newton@linaro.org>
>
>         * benchtests/Makefile (CFLAGS-bench-ffs.c): Add
>         -fno-builtin.  (CFLAGS-bench-ffsll.c): Likewise.
> ---
>  benchtests/Makefile | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/benchtests/Makefile b/benchtests/Makefile
> index 5aa2eb1..525ab09 100644
> --- a/benchtests/Makefile
> +++ b/benchtests/Makefile
> @@ -37,6 +37,9 @@ stdlib-bench := strtod
>
>  benchset := $(string-bench-all) $(stdlib-bench)
>
> +CFLAGS-bench-ffs.c += -fno-builtin
> +CFLAGS-bench-ffsll.c += -fno-builtin
> +
>  LDLIBS-bench-acos = -lm
>  LDLIBS-bench-acosh = -lm
>  LDLIBS-bench-asin = -lm
> --
> 1.8.1.4
>
diff mbox

Patch

diff --git a/benchtests/Makefile b/benchtests/Makefile
index 5aa2eb1..525ab09 100644
--- a/benchtests/Makefile
+++ b/benchtests/Makefile
@@ -37,6 +37,9 @@  stdlib-bench := strtod
 
 benchset := $(string-bench-all) $(stdlib-bench)
 
+CFLAGS-bench-ffs.c += -fno-builtin
+CFLAGS-bench-ffsll.c += -fno-builtin
+
 LDLIBS-bench-acos = -lm
 LDLIBS-bench-acosh = -lm
 LDLIBS-bench-asin = -lm