FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

x86: Remove 32-bit sign extension in offset_in_range

Browse source 
When encoding a 32-bit offset, there is no need to sign-extend it to 64
bits since only the lower 32 bits are used.

	* config/tc-i386.c (offset_in_range): Remove 32-bit sign
	extension.
This commit is contained in:
H.J. Lu 2020-07-13 10:18:39 -07:00
parent f413dd7439
commit 7a70531559
2 changed files with 5 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
gas/ChangeLog
View file

@ -1,3 +1,8 @@
2020-07-13 H.J. Lu <hongjiu.lu@intel.com>
* config/tc-i386.c (offset_in_range): Remove 32-bit sign
extension.
2020-07-13 Nick Clifton <nickc@redhat.com>
* po/fr.po: Updated French translation.

8
gas/config/tc-i386.c
View file

@ -2539,14 +2539,6 @@ offset_in_range (offsetT val, int size)
default: abort ();
}
#ifdef BFD64
/* If BFD64, sign extend val for 32bit address mode. */
if (flag_code != CODE_64BIT
|| i.prefix[ADDR_PREFIX])
if ((val & ~(((addressT) 2 << 31) - 1)) == 0)
val = (val ^ ((addressT) 1 << 31)) - ((addressT) 1 << 31);
#endif
if ((val & ~mask) != 0 && (val & ~mask) != ~mask)
{
char buf1[40], buf2[40];