linux-mips
[Top] [All Lists]

Re: [PATCH] gcc 3.4 drops "accum" clobber, replace with "hi" in time.c

To: <linux-mips@linux-mips.org>
Subject: Re: [PATCH] gcc 3.4 drops "accum" clobber, replace with "hi" in time.c
From: "Bradley D. LaRonde" <brad@laronde.org>
Date: Mon, 12 Apr 2004 20:53:03 -0400
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <Pine.GSO.4.10.10404122244110.8735-100000@helios.et.put.poznan.pl> <20040412231309.GA702@linux-mips.org> <03f301c420e7$d8de2d70$8d01010a@prefect>
Sender: linux-mips-bounce@linux-mips.org
Uh oh, with this patch:

...
time.c: In function `fixed_rate_gettimeoffset':
time.c:242: error: can't find a register in class `HI_REG' while reloading
`asm'
...

Regards,
Brad

----- Original Message ----- 
From: "Bradley D. LaRonde" <brad@laronde.org>
To: <linux-mips@linux-mips.org>
Sent: Monday, April 12, 2004 7:42 PM
Subject: [PATCH] gcc 3.4 drops "accum" clobber, replace with "hi" in time.c


> This 2.4 patch look OK?
>
> Regards,
> Brad
>
> diff -u -r1.1.1.1 time.c
> --- arch/mips/kernel/time.c     10 Nov 2003 21:06:52 -0000      1.1.1.1
> +++ arch/mips/kernel/time.c     12 Apr 2004 23:41:38 -0000
> @@ -242,7 +242,7 @@
>         __asm__("multu  %1,%2"
>                 : "=h" (res)
>                 : "r" (count), "r" (sll32_usecs_per_cycle)
> -               : "lo", "accum");
> +               : "lo", "hi");
>
>         /*
>          * Due to possible jiffies inconsistencies, we need to check
> @@ -297,7 +297,7 @@
>         __asm__("multu  %1,%2"
>                 : "=h" (res)
>                 : "r" (count), "r" (quotient)
> -               : "lo", "accum");
> +               : "lo", "hi");
>
>         /*
>          * Due to possible jiffies inconsistencies, we need to check
> @@ -339,7 +339,7 @@
>                                 : "r" (timerhi), "m" (timerlo),
>                                   "r" (tmp), "r" (USECS_PER_JIFFY),
>                                   "r" (USECS_PER_JIFFY_FRAC)
> -                               : "hi", "lo", "accum");
> +                               : "hi", "lo", "hi");
>                         cached_quotient = quotient;
>                 }
>         }
> @@ -353,7 +353,7 @@
>         __asm__("multu  %1,%2"
>                 : "=h" (res)
>                 : "r" (count), "r" (quotient)
> -               : "lo", "accum");
> +               : "lo", "hi");
>
>         /*
>          * Due to possible jiffies inconsistencies, we need to check
>
>
>


<Prev in Thread] Current Thread [Next in Thread>