diff mbox

target-mips: add ULL suffix in bitswap to avoid compiler warning

Message ID 1413982829-27225-1-git-send-email-leon.alrae@imgtec.com
State New
Headers show

Commit Message

Leon Alrae Oct. 22, 2014, 1 p.m. UTC 
Signed-off-by: Leon Alrae <leon.alrae@imgtec.com>
---
 target-mips/op_helper.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Peter Maydell Oct. 24, 2014, 1:27 p.m. UTC | #1 
On 22 October 2014 14:00, Leon Alrae <leon.alrae@imgtec.com> wrote:
> Signed-off-by: Leon Alrae <leon.alrae@imgtec.com>
> ---
>  target-mips/op_helper.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/target-mips/op_helper.c b/target-mips/op_helper.c
> index 5204ed8..9ec548c 100644
> --- a/target-mips/op_helper.c
> +++ b/target-mips/op_helper.c
> @@ -267,12 +267,12 @@ target_ulong helper_mulshiu(CPUMIPSState *env, target_ulong arg1,
>
>  static inline target_ulong bitswap(target_ulong v)
>  {
> -    v = ((v >> 1) & (target_ulong)0x5555555555555555) |
> -              ((v & (target_ulong)0x5555555555555555) << 1);
> -    v = ((v >> 2) & (target_ulong)0x3333333333333333) |
> -              ((v & (target_ulong)0x3333333333333333) << 2);
> -    v = ((v >> 4) & (target_ulong)0x0F0F0F0F0F0F0F0F) |
> -              ((v & (target_ulong)0x0F0F0F0F0F0F0F0F) << 4);
> +    v = ((v >> 1) & (target_ulong)0x5555555555555555ULL) |
> +              ((v & (target_ulong)0x5555555555555555ULL) << 1);
> +    v = ((v >> 2) & (target_ulong)0x3333333333333333ULL) |
> +              ((v & (target_ulong)0x3333333333333333ULL) << 2);
> +    v = ((v >> 4) & (target_ulong)0x0F0F0F0F0F0F0F0FULL) |
> +              ((v & (target_ulong)0x0F0F0F0F0F0F0F0FULL) << 4);
>      return v;
>  }

Applied to master, thanks.

-- PMM
diff mbox

Patch

diff --git a/target-mips/op_helper.c b/target-mips/op_helper.c
index 5204ed8..9ec548c 100644
--- a/target-mips/op_helper.c
+++ b/target-mips/op_helper.c
@@ -267,12 +267,12 @@  target_ulong helper_mulshiu(CPUMIPSState *env, target_ulong arg1,
 
 static inline target_ulong bitswap(target_ulong v)
 {
-    v = ((v >> 1) & (target_ulong)0x5555555555555555) |
-              ((v & (target_ulong)0x5555555555555555) << 1);
-    v = ((v >> 2) & (target_ulong)0x3333333333333333) |
-              ((v & (target_ulong)0x3333333333333333) << 2);
-    v = ((v >> 4) & (target_ulong)0x0F0F0F0F0F0F0F0F) |
-              ((v & (target_ulong)0x0F0F0F0F0F0F0F0F) << 4);
+    v = ((v >> 1) & (target_ulong)0x5555555555555555ULL) |
+              ((v & (target_ulong)0x5555555555555555ULL) << 1);
+    v = ((v >> 2) & (target_ulong)0x3333333333333333ULL) |
+              ((v & (target_ulong)0x3333333333333333ULL) << 2);
+    v = ((v >> 4) & (target_ulong)0x0F0F0F0F0F0F0F0FULL) |
+              ((v & (target_ulong)0x0F0F0F0F0F0F0F0FULL) << 4);
     return v;
 }