[llvm-commits] [llvm] r79688 - /llvm/trunk/lib/Support/APFloat.cpp

[Eric Christopher]

On Aug 21, 2009, at 4:09 PM, Anton Korobeynikov wrote:

> Author: asl
> Date: Fri Aug 21 18:09:47 2009
> New Revision: 79688
>
> URL: http://llvm.org/viewvc/llvm-project?rev=79688&view=rev
> Log:
> Fix typo.
>
> Modified:
>    llvm/trunk/lib/Support/APFloat.cpp
>
> Modified: llvm/trunk/lib/Support/APFloat.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/APFloat.cpp?rev=79688&r1=79687&r2=79688&view=diff
>
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- llvm/trunk/lib/Support/APFloat.cpp (original)
> +++ llvm/trunk/lib/Support/APFloat.cpp Fri Aug 21 18:09:47 2009
> @@ -2740,7 +2740,7 @@
>   words[0] = mysignificand;
>   words[1] = ((uint64_t)(sign & 1) << 63) |
>              ((myexponent & 0x7fff) << 48) |
> -             (mysignificand & 0xffffffffffffLL);
> +             (mysignificand2 & 0xffffffffffffLL);

Aiee. Any time we have mysignificand and mysignificand2 we need  
renaming of at least one of them. (Anything with "my" should be  
renamed as a matter of course :)

-eric