Jan Beulich JBeulich@suse.com
Wed Jan 10 09:32:00 GMT 2018

• Previous message (by thread): [PATCH] RISC-V: Disassemble x0 based addresses as 0.
• Next message (by thread): Ping: [PATCH] objcopy: fix 32-bit build
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

---

I do assume this isn't sufficient to address the non-64-bit-BFD case as
well, but it's not entirely clear what the right approach to deal with
that would be. I also wonder whether it was really meant to have 59 bits
set here, rather than 63.

binutils/
2018-01-10  Jan Beulich  <jbeulich@suse.com>

	* objcopy.c (merge_gnu_build_notes): Replace 64-bit constants by
	equivalent expressions.

--- a/binutils/objcopy.c
+++ b/binutils/objcopy.c
@@ -2064,7 +2064,7 @@ merge_gnu_build_notes (bfd * abfd, asect
 	     For now though, since v1 and v2 was not intended to
 	     handle gaps, we chose an artificially large end
 	     address.  */
-	  end = (bfd_vma) 0x7ffffffffffffffUL;
+	  end = ~((bfd_vma)1 << 63);
 	  break;
 	  
 	case 8:
@@ -2083,7 +2083,7 @@ merge_gnu_build_notes (bfd * abfd, asect
 		 For now though, since v1 and v2 was not intended to
 		 handle gaps, we chose an artificially large end
 		 address.  */
-	      end = (bfd_vma) 0x7ffffffffffffffUL;
+	      end = ~((bfd_vma)1 << 63);
 	    }
 	  break;
 

---

• Previous message (by thread): [PATCH] RISC-V: Disassemble x0 based addresses as 0.
• Next message (by thread): Ping: [PATCH] objcopy: fix 32-bit build
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

---

More information about the Binutils mailing list