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]

Unreviewed patch: fix some comments


This patch, which fixes typos in some comments, has been unreivewed
since April 25.

2001-05-27  Lars Brinkhoff  <lars@nocrew.org>

	* combine.c (try_combine): Fix typo in comment.
	* expr.c (expand_expr): Likewise.
	* rtl.def (CONST_DOUBLE): There may be 5 'w's.
	* config/i370/i370.md (untyped_call): Fix typos in comment.

Index: combine.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/combine.c,v
retrieving revision 1.205
diff -c -3 -p -r1.205 combine.c
*** combine.c	2001/05/22 07:40:24	1.205
--- combine.c	2001/05/27 08:58:38
*************** cant_combine_insn_p (insn)
*** 1458,1464 ****
     Here I1 and I2 appear earlier than I3.
     I1 can be zero; then we combine just I2 into I3.
  
!    It we are combining three insns and the resulting insn is not recognized,
     try splitting it into two insns.  If that happens, I2 and I3 are retained
     and I1 is pseudo-deleted by turning it into a NOTE.  Otherwise, I1 and I2
     are pseudo-deleted.
--- 1458,1464 ----
     Here I1 and I2 appear earlier than I3.
     I1 can be zero; then we combine just I2 into I3.
  
!    If we are combining three insns and the resulting insn is not recognized,
     try splitting it into two insns.  If that happens, I2 and I3 are retained
     and I1 is pseudo-deleted by turning it into a NOTE.  Otherwise, I1 and I2
     are pseudo-deleted.
Index: expr.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/expr.c,v
retrieving revision 1.319
diff -c -3 -p -r1.319 expr.c
*** expr.c	2001/05/22 07:40:25	1.319
--- expr.c	2001/05/27 08:58:41
*************** expand_expr (exp, target, tmode, modifie
*** 6344,6350 ****
  			       copy_rtx (XEXP (DECL_RTL (exp), 0)));
  
        /* If we got something, return it.  But first, set the alignment
! 	 the address is a register.  */
        if (temp != 0)
  	{
  	  if (GET_CODE (temp) == MEM && GET_CODE (XEXP (temp, 0)) == REG)
--- 6351,6357 ----
  			       copy_rtx (XEXP (DECL_RTL (exp), 0)));
  
        /* If we got something, return it.  But first, set the alignment
! 	 if the address is a register.  */
        if (temp != 0)
  	{
  	  if (GET_CODE (temp) == MEM && GET_CODE (XEXP (temp, 0)) == REG)
Index: rtl.def
===================================================================
RCS file: /cvs/gcc/gcc/gcc/rtl.def,v
retrieving revision 1.44
diff -c -3 -p -r1.44 rtl.def
*** rtl.def	2001/03/28 11:03:56	1.44
--- rtl.def	2001/05/27 08:58:42
*************** DEF_RTL_EXPR(CONST_INT, "const_int", "w"
*** 566,572 ****
     Operand 1 ('0') is a chain of all CONST_DOUBLEs in use in the
     current function.
     Remaining operands hold the actual value.  They are all 'w' and
!    there may be from 1 to 4; see rtl.c.  */
  DEF_RTL_EXPR(CONST_DOUBLE, "const_double", CONST_DOUBLE_FORMAT, 'o')
  
  /* String constant.  Used only for attributes right now.  */
--- 566,572 ----
     Operand 1 ('0') is a chain of all CONST_DOUBLEs in use in the
     current function.
     Remaining operands hold the actual value.  They are all 'w' and
!    there may be from 1 to 5; see rtl.c.  */
  DEF_RTL_EXPR(CONST_DOUBLE, "const_double", CONST_DOUBLE_FORMAT, 'o')
  
  /* String constant.  Used only for attributes right now.  */
Index: config/i370/i370.md
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/i370/i370.md,v
retrieving revision 1.16
diff -c -3 -p -r1.16 i370.md
*** i370.md	2001/04/03 15:05:43	1.16
--- i370.md	2001/05/27 08:58:43
*************** check_label_emit ();
*** 4703,4710 ****
  ;; Call subroutine returning any type.
  ;; This instruction pattern appears to be used only by the
  ;; expand_builtin_apply definition for __builtin_apply.  It is needed
! ;; since call_value might return an in in r15 or a float in fpr0 (r16)
! ;; and the builtin code calla abort since the reg is ambiguous. Well,
  ;; the below is probably broken anyway, we just want to go for now.
  ;;
  (define_expand "untyped_call"
--- 4703,4710 ----
  ;; Call subroutine returning any type.
  ;; This instruction pattern appears to be used only by the
  ;; expand_builtin_apply definition for __builtin_apply.  It is needed
! ;; since call_value might return an int in r15 or a float in fpr0 (r16)
! ;; and the builtin code calls abort since the reg is ambiguous. Well,
  ;; the below is probably broken anyway, we just want to go for now.
  ;;
  (define_expand "untyped_call"


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