This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[m68k patch] update and fix comments in m68k.c


Hello!

The following patch fixes comment formatting at several places and
moves one comment to its new place.

Gunther

-- cut --
2003-10-21  Gunther Nikl  <gni@gecko.de>

	* config/m68k/m68k.c (m68k_output_function_prologue): Move front
	comment from here to...
	(m68k_save_reg): Here. Fix comment formatting.
	(m68k_output_function_prologue): Fix comment formatting.
	(m68k_output_function_epilogue): Likewise.
	(const_method): Likewise.

--- m68k.c.orig	Tue Oct 21 13:32:35 2003
+++ m68k.c	Tue Oct 21 14:37:52 2003
@@ -392,7 +392,12 @@ m68k_initial_elimination_offset (int fro
   abort();
 }
 
-/* Return true if we need to save REGNO. */
+/* Refer to the array `regs_ever_live' to determine which registers
+   to save; `regs_ever_live[I]' is nonzero if register number I
+   is ever used in the function.  This function is responsible for
+   knowing which registers should not be saved even if used.
+   Return true if we need to save REGNO.  */
+
 static bool
 m68k_save_reg (unsigned int regno, bool interrupt_handler)
 {
@@ -442,15 +447,7 @@ m68k_save_reg (unsigned int regno, bool 
 
 /* This function generates the assembly code for function entry.
    STREAM is a stdio stream to output the code to.
-   SIZE is an int: how many units of temporary storage to allocate.
-   Refer to the array `regs_ever_live' to determine which registers
-   to save; `regs_ever_live[I]' is nonzero if register number I
-   is ever used in the function.  This function is responsible for
-   knowing which registers should not be saved even if used.  */
-
-
-/* Note that the order of the bit mask for fmovem is the opposite
-   of the order for movem!  */
+   SIZE is an int: how many units of temporary storage to allocate.  */
 
 static void
 m68k_output_function_prologue (FILE *stream, HOST_WIDE_INT size ATTRIBUTE_UNUSED)
@@ -475,7 +472,7 @@ m68k_output_function_prologue (FILE *str
 #endif
     }
 
-  /* on ColdFire add register save into initial stack frame setup, if possible */
+  /* On ColdFire add register save into initial stack frame setup, if possible.  */
   num_saved_regs = 0;
   if (TARGET_COLDFIRE && current_frame.reg_no > 2)
     num_saved_regs = current_frame.reg_no;
@@ -657,7 +654,7 @@ m68k_output_function_prologue (FILE *str
       /* Store each separately in the same order moveml uses.
          Using two movel instructions instead of a single moveml
          is about 15% faster for the 68020 and 68030 at no expense
-         in code size */
+         in code size.  */
 
       int i;
 
@@ -803,14 +800,14 @@ m68k_output_function_epilogue (FILE *str
 	     || (! current_function_calls_alloca && leaf_function_p ());
 
   /* fsize_with_regs is the size we need to adjust the sp when
-     popping the frame */
+     popping the frame.  */
   fsize_with_regs = fsize;
 
   /* Because the ColdFire doesn't support moveml with
      complex address modes, we must adjust the stack manually
      after restoring registers. When the frame pointer isn't used,
      we can merge movem adjustment into frame unlinking
-     made immediately after it. */
+     made immediately after it.  */
   if (TARGET_COLDFIRE && restore_from_sp && (current_frame.reg_no > 2))
     fsize_with_regs += current_frame.reg_no * 4;
 
@@ -819,7 +816,7 @@ m68k_output_function_epilogue (FILE *str
       && (current_frame.reg_mask || current_frame.fpu_mask))
     {
       /* Because the ColdFire doesn't support moveml with
-         complex address modes we make an extra correction here */
+         complex address modes we make an extra correction here.  */
       if (TARGET_COLDFIRE)
         {
 #ifdef MOTOROLA
@@ -897,7 +894,7 @@ m68k_output_function_epilogue (FILE *str
     }
   else if (current_frame.reg_mask)
     {
-      /* The ColdFire requires special handling due to its limited moveml insn */
+      /* The ColdFire requires special handling due to its limited moveml insn.  */
       if (TARGET_COLDFIRE)
         {
           if (big)
@@ -1662,7 +1659,7 @@ const_method (rtx constant)
     return MOVQ;
 
   /* The ColdFire doesn't have byte or word operations.  */
-  /* FIXME: This may not be useful for the m68060 either */
+  /* FIXME: This may not be useful for the m68060 either.  */
   if (!TARGET_COLDFIRE) 
     {
       /* if -256 < N < 256 but N is not in range for a moveq
-- cut --


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]