]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
gas md_apply_fix value casts
authorAlan Modra <amodra@gmail.com>
Mon, 9 Jun 2025 07:30:48 +0000 (17:00 +0930)
committerAlan Modra <amodra@gmail.com>
Tue, 10 Jun 2025 22:15:18 +0000 (07:45 +0930)
These are all innocuous but unneeded.  pdp11 and ppc are only formatting.

gas/config/tc-cris.c
gas/config/tc-metag.c
gas/config/tc-microblaze.c
gas/config/tc-mmix.c
gas/config/tc-mn10300.c
gas/config/tc-pdp11.c
gas/config/tc-ppc.c
gas/config/tc-rl78.c
gas/config/tc-rx.c
gas/config/tc-sh.c
gas/config/tc-wasm32.c

index 2ceac0ed201331c7a3dcd0652497198a80a1f822..5780386e585ef119f59a819dd2094b9ce92665d1 100644 (file)
@@ -4039,9 +4039,8 @@ void
 md_apply_fix (fixS *fixP, valueT *valP, segT seg)
 {
   /* This assignment truncates upper bits if valueT is 64 bits (as with
-     --enable-64-bit-bfd), which is fine here, though we cast to avoid
-     any compiler warnings.  */
-  long val = (long) *valP;
+     --enable-64-bit-bfd), which is fine here.  */
+  long val = *valP;
   char *buf = fixP->fx_where + fixP->fx_frag->fr_literal;
 
   if (fixP->fx_addsy == 0 && !fixP->fx_pcrel)
index cd0c50ff9c990cb34d75e65e5105cdf4e1f9238a..bb8e897b57a58990f5d7b1192f8dbd53bf702271 100644 (file)
@@ -7029,7 +7029,7 @@ void
 md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
 {
   char *buf = fixP->fx_where + fixP->fx_frag->fr_literal;
-  int value = (int)*valP;
+  int value = *valP;
 
   switch (fixP->fx_r_type)
     {
index 3bf2ac16fe755aedaa7453306dcdaa7771249d9b..917b8f64c8005e133460d466d7a4acbdbf86bb9a 100644 (file)
@@ -2042,7 +2042,7 @@ md_apply_fix (fixS *   fixP,
   const char *       file = fixP->fx_file ? fixP->fx_file : _("unknown");
   const char * symname;
   /* Note: use offsetT because it is signed, valueT is unsigned.  */
-  offsetT      val  = (offsetT) * valp;
+  offsetT      val  = *valp;
   int          i;
   struct op_code_struct * opcode1;
   unsigned long inst1;
index eb29b14d1ae2d5b9a9f68b7f40c32c135af72cc3..dea91816b0fb9722fe0e0a3211254e1b7ae4403c 100644 (file)
@@ -2426,12 +2426,11 @@ md_convert_frag (bfd *abfd ATTRIBUTE_UNUSED, segT sec ATTRIBUTE_UNUSED,
 void
 md_apply_fix (fixS *fixP, valueT *valP, segT segment)
 {
-  char *buf  = fixP->fx_where + fixP->fx_frag->fr_literal;
+  char *buf = fixP->fx_where + fixP->fx_frag->fr_literal;
   /* Note: use offsetT because it is signed, valueT is unsigned.  */
-  offsetT val  = (offsetT) * valP;
-  segT symsec
-    = (fixP->fx_addsy == NULL
-       ? absolute_section : S_GET_SEGMENT (fixP->fx_addsy));
+  offsetT val = *valP;
+  segT symsec = (fixP->fx_addsy == NULL
+                ? absolute_section : S_GET_SEGMENT (fixP->fx_addsy));
 
   /* If the fix is relative to a symbol which is not defined, or, (if
      pcrel), not in the same segment as the fix, we cannot resolve it
index 55747e0280a919b05a621d0fb1ab2d63b6b730be..c4cf804005527b302babe12002163dd88025ba87 100644 (file)
@@ -2325,7 +2325,7 @@ md_apply_fix (fixS * fixP, valueT * valP, segT seg)
 {
   char * fixpos = fixP->fx_where + fixP->fx_frag->fr_literal;
   int size = 0;
-  int value = (int) * valP;
+  int value = *valP;
 
   gas_assert (fixP->fx_r_type < BFD_RELOC_UNUSED);
 
index 7566bbbcbff6fa18814d0272852488df3883a58f..2c8a1b891c90bb083c5d656ad56c67b102df66a9 100644 (file)
@@ -239,12 +239,12 @@ md_number_to_chars (char con[], valueT value, int nbytes)
 
 void
 md_apply_fix (fixS *fixP,
-              valueT * valP,
-              segT seg ATTRIBUTE_UNUSED)
+             valueT *valP,
+             segT seg ATTRIBUTE_UNUSED)
 {
   valueT code;
   valueT mask;
-  valueT val = * valP;
+  valueT val = *valP;
   char *buf;
   int shift;
   int size;
index 43dddb0d9075ae3383409eb62246c9af61b2f96f..714dfca4670353a17d634853e9d4c3a22ede6343 100644 (file)
@@ -6964,7 +6964,7 @@ ppc_handle_align (segT sec, struct frag *fragP)
 void
 md_apply_fix (fixS *fixP, valueT *valP, segT seg)
 {
-  valueT value = * valP;
+  valueT value = *valP;
   offsetT fieldval;
   const struct powerpc_operand *operand;
 
index 6d24bc3ecb565bd964f5f7ed12940e3523988826..3593b00fb969e0efc3f578cb3aa809f39a2508c1 100644 (file)
@@ -1420,7 +1420,7 @@ md_apply_fix (struct fix * f ATTRIBUTE_UNUSED,
     return;
 
   op = f->fx_frag->fr_literal + f->fx_where;
-  val = (unsigned long) * t;
+  val = *t;
 
   if (f->fx_addsy == NULL)
     f->fx_done = 1;
index bd45c53cfa0545a558c877d59160db89f64d2bf1..0ecda6411e8fe03c4d3556660c0ada914d4980d8 100644 (file)
@@ -2256,9 +2256,9 @@ rx_cons_fix_new (fragS *  frag,
 }
 
 void
-md_apply_fix (struct fix * f ATTRIBUTE_UNUSED,
-             valueT *     t ATTRIBUTE_UNUSED,
-             segT         s ATTRIBUTE_UNUSED)
+md_apply_fix (struct fix *f,
+             valueT *t,
+             segT s ATTRIBUTE_UNUSED)
 {
   /* Instruction bytes are always little endian.  */
   char * op;
@@ -2274,7 +2274,7 @@ md_apply_fix (struct fix * f ATTRIBUTE_UNUSED,
 #define OP4(x) op[target_big_endian ? 3-x : x]
 
   op = f->fx_frag->fr_literal + f->fx_where;
-  val = (unsigned long) * t;
+  val = *t;
 
   /* Opcode words are always the same endian.  Data words are either
      big or little endian.  */
index 0f890c98cc9704d8f75a1b07f18103aa4bb7f1c7..9620eef9e936a2b4441ee266844130cb40f372e8 100644 (file)
@@ -3396,7 +3396,7 @@ md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
   char *buf = fixP->fx_where + fixP->fx_frag->fr_literal;
   int lowbyte = target_big_endian ? 1 : 0;
   int highbyte = target_big_endian ? 0 : 1;
-  long val = (long) *valP;
+  long val = *valP;
   long max, min;
   int shift;
 
index ea47a02bd0c76138ac99326e6c9eb9584c5e8080..066255f899450f6abe0a7c75a98edd35a96c87b4 100644 (file)
@@ -204,7 +204,7 @@ void
 md_apply_fix (fixS * fixP, valueT * valP, segT seg ATTRIBUTE_UNUSED)
 {
   char *buf = fixP->fx_where + fixP->fx_frag->fr_literal;
-  long val = (long) *valP;
+  valueT val = *valP;
 
   if (fixP->fx_pcrel)
     {