* Claudiu Zissulescu <claudiu.zissule...@synopsys.com> [2016-07-08 13:41:23 
+0200]:

> Don't use CPU macros, use CPU feature macros.
> 
> libgcc/
> 2016-05-26  Claudiu Zissulescu  <claz...@synopsys.com>
> 
>       * config/arc/lib1funcs.S (__mulsi3): Use feature defines instead
>       of checking for cpus.
>       (__umulsidi3, __umulsi3_highpart, __udivmodsi4, __divsi3)
>       (__modsi3, __clzsi2): Likewise.
> ---
>  libgcc/config/arc/lib1funcs.S | 45 
> +++++++++++++++++++++++--------------------
>  1 file changed, 24 insertions(+), 21 deletions(-)
> 
> diff --git a/libgcc/config/arc/lib1funcs.S b/libgcc/config/arc/lib1funcs.S
> index 9bb25e0..422fd95 100644
> --- a/libgcc/config/arc/lib1funcs.S
> +++ b/libgcc/config/arc/lib1funcs.S
> @@ -188,18 +189,18 @@ SYM(__umulsi3_highpart):
>  */
>  #include "ieee-754/arc-ieee-754.h"
>  
> -#ifdef __ARC700__
> +#ifdef __ARC_MPY__
>       mov_s   r12,DBL0L
>       mpyu    DBL0L,r12,DBL0H
>       j_s.d   [blink]
> -     mpyhu   DBL0H,r12,DBL0H
> +     MPYHU   DBL0H,r12,DBL0H

Is there a reason that instruction should be uppercase?

This otherwise looks fine to me.

Thanks,
Andrew

Reply via email to