This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[committed, PATCH 1/2] Use 0xffffffffffffffffLL to silence GCC 6 warning


GCC 6 warns:

error: result of â4294967295ll << 32â requires 65 bits to represent, but âlong long intâ only has 64 bits [-Werror=shift-overflow=]

on ((((bfd_signed_vma) 0xffffffff) << 32) | 0xffffffff).  This patch
replaces it with bfd_signed_vma) 0xffffffffffffffffLL.

	* stabs.c (parse_stab_range_type): Use 0xffffffffffffffffLL.
---
 binutils/ChangeLog | 4 ++++
 binutils/stabs.c   | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 4679498..bbaf937 100644
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,7 @@
+2015-07-24  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* stabs.c (parse_stab_range_type): Use 0xffffffffffffffffLL.
+
 2015-07-22  Alan Modra  <amodra@gmail.com>
 
 	* readelf.c (dump_relocations): Show MIN_INT addends as negative.
diff --git a/binutils/stabs.c b/binutils/stabs.c
index 4f17308..5b4c8a3 100644
--- a/binutils/stabs.c
+++ b/binutils/stabs.c
@@ -1774,7 +1774,7 @@ parse_stab_range_type (void *dhandle, struct stab_handle *info, const char *type
 	  else if (n3 == (bfd_signed_vma) 0xffffffff)
 	    return debug_make_int_type (dhandle, 4, TRUE);
 #ifdef BFD64
-	  else if (n3 == ((((bfd_signed_vma) 0xffffffff) << 32) | 0xffffffff))
+	  else if (n3 == (bfd_signed_vma) 0xffffffffffffffffLL)
 	    return debug_make_int_type (dhandle, 8, TRUE);
 #endif
 	}
-- 
2.4.3


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]