[binutils-gdb] x86: tighten extend-to-32bit-address conditions

Jan Beulich jbeulich@sourceware.org
Fri May 19 07:19:21 GMT 2023


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=df81d460b2d9ec7327a0c1ab2344d7ec62874ce0

commit df81d460b2d9ec7327a0c1ab2344d7ec62874ce0
Author: Jan Beulich <jbeulich@suse.com>
Date:   Fri May 19 09:14:40 2023 +0200

    x86: tighten extend-to-32bit-address conditions
    
    In a442cac5084e ("ix86: wrap constants") I made the truncation condition
    too relaxed: Any indication of a mode that's possible with BFD64 only
    should avoid the truncation. Therefore, like in the other two cases of
    calls to extend_to_32bit_address(), also check whether we're generating
    a 64-bit object.

Diff:
---
 gas/config/tc-i386.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index c0bf4fc0c4e..9efb1e58447 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -11684,7 +11684,7 @@ i386_finalize_immediate (segT exp_seg ATTRIBUTE_UNUSED, expressionS *exp,
 
       /* If not 64bit, sign/zero extend val, to account for wraparound
 	 when !BFD64.  */
-      if (flag_code != CODE_64BIT)
+      if (flag_code != CODE_64BIT && !object_64bit)
 	exp->X_add_number = extend_to_32bit_address (exp->X_add_number);
     }
 #if (defined (OBJ_AOUT) || defined (OBJ_MAYBE_AOUT))
@@ -11976,7 +11976,7 @@ i386_finalize_displacement (segT exp_seg ATTRIBUTE_UNUSED, expressionS *exp,
 
 	 If not 64bit, sign/zero extend val, to account for wraparound
 	 when !BFD64.  */
-      if (flag_code != CODE_64BIT)
+      if (flag_code != CODE_64BIT && !object_64bit)
 	exp->X_add_number = extend_to_32bit_address (exp->X_add_number);
     }


More information about the Binutils-cvs mailing list