From: Alan Modra <amodra@gmail.com>
Date: Thu, 9 May 2013 04:01:53 +0000 (+0000)
Subject: 	* config/tc-ppc.c (md_apply_fix): Sign extend fieldval under
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=da8094d7964e634fb0ec5632d36f9bd282f3170e;p=binutils-gdb.git

	* config/tc-ppc.c (md_apply_fix): Sign extend fieldval under
	control of operand flag bits.
---

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 9ec5ff175b8..b32779d4e0b 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2013-05-09  Alan Modra  <amodra@gmail.com>
+
+	* config/tc-ppc.c (md_apply_fix): Sign extend fieldval under
+	control of operand flag bits.
+
 2013-05-07  Alan Modra  <amodra@gmail.com>
 
 	* config/tc-ppc.c (PPC_VLE_SPLIT16A): Delete unused macro.
diff --git a/gas/config/tc-ppc.c b/gas/config/tc-ppc.c
index 9baf34d74b4..904166e88d4 100644
--- a/gas/config/tc-ppc.c
+++ b/gas/config/tc-ppc.c
@@ -6369,7 +6369,10 @@ md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
 	case BFD_RELOC_LO16_PCREL:
 	case BFD_RELOC_PPC_VLE_LO16A:
 	case BFD_RELOC_PPC_VLE_LO16D:
-	  fieldval = SEX16 (value);
+	  fieldval = value & 0xffff;
+	sign_extend_16:
+	  if ((operand->flags & PPC_OPERAND_SIGNED) != 0)
+	    fieldval = (fieldval ^ 0x8000) - 0x8000;
 	  fixP->fx_no_overflow = 1;
 	  break;
 
@@ -6380,9 +6383,8 @@ md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
 	case BFD_RELOC_HI16_PCREL:
 	case BFD_RELOC_PPC_VLE_HI16A:
 	case BFD_RELOC_PPC_VLE_HI16D:
-	  fieldval = SEX16 (PPC_HI (value));
-	  fixP->fx_no_overflow = 1;
-	  break;
+	  fieldval = PPC_HI (value);
+	  goto sign_extend_16;
 
 	case BFD_RELOC_HI16_S:
 	  if (fixP->fx_pcrel)
@@ -6391,38 +6393,33 @@ md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
 	case BFD_RELOC_HI16_S_PCREL:
 	case BFD_RELOC_PPC_VLE_HA16A:
 	case BFD_RELOC_PPC_VLE_HA16D:
-	  fieldval = SEX16 (PPC_HA (value));
-	  fixP->fx_no_overflow = 1;
-	  break;
+	  fieldval = PPC_HA (value);
+	  goto sign_extend_16;
 
 #ifdef OBJ_ELF
 	case BFD_RELOC_PPC64_HIGHER:
 	  if (fixP->fx_pcrel)
 	    goto bad_pcrel;
-	  fieldval = SEX16 (PPC_HIGHER (value));
-	  fixP->fx_no_overflow = 1;
-	  break;
+	  fieldval = PPC_HIGHER (value);
+	  goto sign_extend_16;
 
 	case BFD_RELOC_PPC64_HIGHER_S:
 	  if (fixP->fx_pcrel)
 	    goto bad_pcrel;
-	  fieldval = SEX16 (PPC_HIGHERA (value));
-	  fixP->fx_no_overflow = 1;
-	  break;
+	  fieldval = PPC_HIGHERA (value);
+	  goto sign_extend_16;
 
 	case BFD_RELOC_PPC64_HIGHEST:
 	  if (fixP->fx_pcrel)
 	    goto bad_pcrel;
-	  fieldval = SEX16 (PPC_HIGHEST (value));
-	  fixP->fx_no_overflow = 1;
-	  break;
+	  fieldval = PPC_HIGHEST (value);
+	  goto sign_extend_16;
 
 	case BFD_RELOC_PPC64_HIGHEST_S:
 	  if (fixP->fx_pcrel)
 	    goto bad_pcrel;
-	  fieldval = SEX16 (PPC_HIGHESTA (value));
-	  fixP->fx_no_overflow = 1;
-	  break;
+	  fieldval = PPC_HIGHESTA (value);
+	  goto sign_extend_16;
 
 	  /* The following relocs can't be calculated by the assembler.
 	     Leave the field zero.  */