gas/

	* config/tc-mips.c (reg_needs_delay): Move later in file.
	Use gpr_write_mask.
	(insns_between): Use gpr_read_mask instead of EXTRACT_OPERAND.

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 0641b1c29d561464e1c7e8ce2dd282ae84cae9c5..e1cab9afeb66fb89be4d2ff9dd6e1ea2820451f9 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,9 @@
+2013-08-01  Richard Sandiford  <[email protected]>
+
+       * config/tc-mips.c (reg_needs_delay): Move later in file.
+       Use gpr_write_mask.
+       (insns_between): Use gpr_read_mask instead of EXTRACT_OPERAND.
+

 2013-07-26  Sergey Guriev  <[email protected]>
            Alexander Ivchenko  <[email protected]>
            Maxim Kuznetsov  <[email protected]>
 2013-07-26  Sergey Guriev  <[email protected]>
            Alexander Ivchenko  <[email protected]>
            Maxim Kuznetsov  <[email protected]>

diff --git a/gas/config/tc-mips.c b/gas/config/tc-mips.c
index 57df9a37227cd8092c2fe59aa673910ba991832b..68241a0024d8435da3bc90fb856074d4fb7d9523 100644 (file)
--- a/gas/config/tc-mips.c
+++ b/gas/config/tc-mips.c
@@ -3397,32 +3397,6 @@ fixup_has_matching_lo_p (fixS *fixp)
          && fixp->fx_offset == fixp->fx_next->fx_offset);
 }
 
          && fixp->fx_offset == fixp->fx_next->fx_offset);
 }
 

-/* This function returns true if modifying a register requires a
-   delay.  */
-
-static int
-reg_needs_delay (unsigned int reg)
-{
-  unsigned long prev_pinfo;
-
-  prev_pinfo = history[0].insn_mo->pinfo;
-  if (! mips_opts.noreorder
-      && (((prev_pinfo & INSN_LOAD_MEMORY_DELAY)
-          && ! gpr_interlocks)
-         || ((prev_pinfo & INSN_LOAD_COPROC_DELAY)
-             && ! cop_interlocks)))
-    {
-      /* A load from a coprocessor or from memory.  All load delays
-        delay the use of general register rt for one instruction.  */
-      /* Itbl support may require additional care here.  */
-      know (prev_pinfo & INSN_WRITE_GPR_T);
-      if (reg == EXTRACT_OPERAND (mips_opts.micromips, RT, history[0]))
-       return 1;
-    }
-
-  return 0;
-}
-

 /* Move all labels in LABELS to the current insertion point.  TEXT_P
    says whether the labels refer to text or data.  */
 
 /* Move all labels in LABELS to the current insertion point.  TEXT_P
    says whether the labels refer to text or data.  */
 


@@ -5081,6 +5055,23 @@ check_completed_insn (struct mips_arg_info *arg)
     }
 }
 
     }
 }
 

+/* Return true if modifying general-purpose register REG needs a delay.  */
+
+static bfd_boolean
+reg_needs_delay (unsigned int reg)
+{
+  unsigned long prev_pinfo;
+
+  prev_pinfo = history[0].insn_mo->pinfo;
+  if (!mips_opts.noreorder
+      && (((prev_pinfo & INSN_LOAD_MEMORY_DELAY) && !gpr_interlocks)
+         || ((prev_pinfo & INSN_LOAD_COPROC_DELAY) && !cop_interlocks))
+      && (gpr_write_mask (&history[0]) & (1 << reg)))
+    return TRUE;
+
+  return FALSE;
+}
+

 /* Classify an instruction according to the FIX_VR4120_* enumeration.
    Return NUM_FIX_VR4120_CLASSES if the instruction isn't affected
    by VR4120 errata.  */
 /* Classify an instruction according to the FIX_VR4120_* enumeration.
    Return NUM_FIX_VR4120_CLASSES if the instruction isn't affected
    by VR4120 errata.  */


@@ -5117,16 +5108,11 @@ insns_between (const struct mips_cl_insn *insn1,
   unsigned long pinfo1, pinfo2;
   unsigned int mask;
 
   unsigned long pinfo1, pinfo2;
   unsigned int mask;
 

-  /* This function needs to know which pinfo flags are set for INSN2
-     and which registers INSN2 uses.  The former is stored in PINFO2 and
-     the latter is tested via INSN2_USES_GPR.  If INSN2 is null, PINFO2
-     will have every flag set and INSN2_USES_GPR will always return true.  */
+  /* If INFO2 is null, pessimistically assume that all flags are set for
+     the second instruction.  */

   pinfo1 = insn1->insn_mo->pinfo;
   pinfo2 = insn2 ? insn2->insn_mo->pinfo : ~0U;
 
   pinfo1 = insn1->insn_mo->pinfo;
   pinfo2 = insn2 ? insn2->insn_mo->pinfo : ~0U;
 

-#define INSN2_USES_GPR(REG) \
-  (insn2 == NULL || (gpr_read_mask (insn2) & (1U << (REG))) != 0)
-

   /* For most targets, write-after-read dependencies on the HI and LO
      registers must be separated by at least two instructions.  */
   if (!hilo_interlocks)
   /* For most targets, write-after-read dependencies on the HI and LO
      registers must be separated by at least two instructions.  */
   if (!hilo_interlocks)


@@ -5142,7 +5128,7 @@ insns_between (const struct mips_cl_insn *insn1,
   if (mips_7000_hilo_fix
       && !mips_opts.micromips
       && MF_HILO_INSN (pinfo1)
   if (mips_7000_hilo_fix
       && !mips_opts.micromips
       && MF_HILO_INSN (pinfo1)

-      && INSN2_USES_GPR (EXTRACT_OPERAND (0, RD, *insn1)))
+      && (insn2 == NULL || (gpr_read_mask (insn2) & gpr_write_mask (insn1))))

     return 2;
 
   /* If we're working around 24K errata, one instruction is required
     return 2;
 
   /* If we're working around 24K errata, one instruction is required


@@ -5185,8 +5171,7 @@ insns_between (const struct mips_cl_insn *insn1,
       if ((!gpr_interlocks && (pinfo1 & INSN_LOAD_MEMORY_DELAY))
          || (!cop_interlocks && (pinfo1 & INSN_LOAD_COPROC_DELAY)))
        {
       if ((!gpr_interlocks && (pinfo1 & INSN_LOAD_MEMORY_DELAY))
          || (!cop_interlocks && (pinfo1 & INSN_LOAD_COPROC_DELAY)))
        {

-         know (pinfo1 & INSN_WRITE_GPR_T);
-         if (INSN2_USES_GPR (EXTRACT_OPERAND (0, RT, *insn1)))
+         if (insn2 == NULL || (gpr_read_mask (insn2) & gpr_write_mask (insn1)))

            return 1;
        }
 
            return 1;
        }
 


@@ -5237,8 +5222,6 @@ insns_between (const struct mips_cl_insn *insn1,
        return 1;
     }
 
        return 1;
     }
 

-#undef INSN2_USES_GPR
-

   return 0;
 }
 
   return 0;
 }