From afa5870f1bfafa1ccfa8a4b3f432c07c19a2447e Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Fri, 24 Jul 2015 09:10:18 -0700 Subject: [PATCH] Add 'U' suffix to silence GCC 6 warning MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit GCC 6 warns: error: result of β€˜63 << 26’ requires 33 bits to represent, but β€˜int’ only has 32 bits [-Werror=shift-overflow=] on 0x3f << 26. This patch adds 'U' suffix to make integer constant unsigned. * alpha.c (OP_Jxx): Add 'U' suffix to make it unsigned. (OP_BSR): Likewise. (Jxx_FUNC_JMP): Likewise. (Jxx_FUNC_JSR): Likewise. (Jxx_FUNC_RET): Likewise. (Jxx_FUNC_JSR_COROUTINE): Likewise. (alpha_find_call): Replace 0x3f with 0x3fU. --- gprof/ChangeLog | 10 ++++++++++ gprof/alpha.c | 14 +++++++------- 2 files changed, 17 insertions(+), 7 deletions(-) diff --git a/gprof/ChangeLog b/gprof/ChangeLog index 7a2087b8f7..d93046b17d 100644 --- a/gprof/ChangeLog +++ b/gprof/ChangeLog @@ -1,3 +1,13 @@ +2015-07-24 H.J. Lu + + * alpha.c (OP_Jxx): Add 'U' suffix to make it unsigned. + (OP_BSR): Likewise. + (Jxx_FUNC_JMP): Likewise. + (Jxx_FUNC_JSR): Likewise. + (Jxx_FUNC_RET): Likewise. + (Jxx_FUNC_JSR_COROUTINE): Likewise. + (alpha_find_call): Replace 0x3f with 0x3fU. + 2015-04-29 Nick Clifton * po/da.po: Update Danish translation. diff --git a/gprof/alpha.c b/gprof/alpha.c index 1a3ebf9704..d84cdf0a91 100644 --- a/gprof/alpha.c +++ b/gprof/alpha.c @@ -37,13 +37,13 @@ /* * Opcodes of the call instructions: */ -#define OP_Jxx 0x1a -#define OP_BSR 0x34 +#define OP_Jxx 0x1aU +#define OP_BSR 0x34U -#define Jxx_FUNC_JMP 0 -#define Jxx_FUNC_JSR 1 -#define Jxx_FUNC_RET 2 -#define Jxx_FUNC_JSR_COROUTINE 3 +#define Jxx_FUNC_JMP 0U +#define Jxx_FUNC_JSR 1U +#define Jxx_FUNC_RET 2U +#define Jxx_FUNC_JSR_COROUTINE 3U /* *INDENT-OFF* */ /* Here to document only. We can't use this when cross compiling as @@ -111,7 +111,7 @@ alpha_find_call (Sym *parent, bfd_vma p_lowpc, bfd_vma p_highpc) { insn = bfd_get_32 (core_bfd, ((unsigned char *) core_text_space + pc - core_text_sect->vma)); - switch (insn & (0x3f << 26)) + switch (insn & (0x3fU << 26)) { case OP_Jxx << 26: /*