On Mon, Apr 24, 2017 at 6:21 PM, Joel Sherrill <[email protected]> wrote:
> Module: rtems
> Branch: master
> Commit: 906e67fe88658122d515a5dcc0570ee56a02877d
> Changeset:
> http://git.rtems.org/rtems/commit/?id=906e67fe88658122d515a5dcc0570ee56a02877d
>
> Author: Joel Sherrill <[email protected]>
> Date: Sun Apr 23 11:50:46 2017 -0500
>
> dhrystone/dhry_1.c: Fix may be used uninitialized warning
>
> ---
>
> testsuites/benchmarks/dhrystone/dhry_1.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/testsuites/benchmarks/dhrystone/dhry_1.c
> b/testsuites/benchmarks/dhrystone/dhry_1.c
> index 36ac303..9b64d3a 100644
> --- a/testsuites/benchmarks/dhrystone/dhry_1.c
> +++ b/testsuites/benchmarks/dhrystone/dhry_1.c
> @@ -143,6 +143,11 @@ execution_start:
> Begin_Time = time ( (long *) 0);
> #endif
>
> +#ifdef __rtems__
> + /* avoid used uninitialized warning */
> + Int_2_Loc = 0;
> +#endif
> +
This looks like unnecessarily catering to the compiler. At any rate,
this kind of change should be located before the call to the timer,
since it maybe perturbs the benchmark timing.
> for (Run_Index = 1; Run_Index <= Number_Of_Runs; ++Run_Index)
> {
>
>
> _______________________________________________
> vc mailing list
> [email protected]
> http://lists.rtems.org/mailman/listinfo/vc
_______________________________________________
devel mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/devel
