Re: [Qemu-devel] [PATCH] target-arm: use clz32() instead of a for loop

[Laurent Desnogues]

On Thu, Oct 15, 2009 at 11:14 PM, Aurelien Jarno <address@hidden> wrote:
> Signed-off-by: Aurelien Jarno <address@hidden>
> ---
>  target-arm/helper.c |    6 ++----
>  1 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/target-arm/helper.c b/target-arm/helper.c
> index 701629a..656b5df 100644
> --- a/target-arm/helper.c
> +++ b/target-arm/helper.c
> @@ -7,6 +7,7 @@
>  #include "gdbstub.h"
>  #include "helpers.h"
>  #include "qemu-common.h"
> +#include "host-utils.h"
>
>  static uint32_t cortexa8_cp15_c0_c1[8] =
>  { 0x1031, 0x11, 0x400, 0, 0x31100003, 0x20000000, 0x01202000, 0x11 };
> @@ -394,10 +395,7 @@ uint32_t HELPER(uxtb16)(uint32_t x)
>
>  uint32_t HELPER(clz)(uint32_t x)
>  {
> -    int count;
> -    for (count = 32; x; count--)
> -        x >>= 1;
> -    return count;
> +    return clz32(x);
>  }
>
>  int32_t HELPER(sdiv)(int32_t num, int32_t den)
> --
> 1.6.1.3

Acked-by: Laurent Desnogues <address@hidden>