This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[committed, PATCH 2/2] Add 'U' suffix to silence GCC 6 warning
- From: "H.J. Lu" <hongjiu dot lu at intel dot com>
- To: binutils at sourceware dot org
- Date: Fri, 24 Jul 2015 09:14:01 -0700
- Subject: [committed, PATCH 2/2] Add 'U' suffix to silence GCC 6 warning
- Authentication-results: sourceware.org; auth=none
- Reply-to: "H.J. Lu" <hjl dot tools at gmail dot com>
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 7a2087b..d93046b 100644
--- a/gprof/ChangeLog
+++ b/gprof/ChangeLog
@@ -1,3 +1,13 @@
+2015-07-24 H.J. Lu <hongjiu.lu@intel.com>
+
+ * 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 <nickc@redhat.com>
* po/da.po: Update Danish translation.
diff --git a/gprof/alpha.c b/gprof/alpha.c
index 1a3ebf9..d84cdf0 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:
/*
--
2.4.3