v850.c (print_operand): Use gcc_assert and gcc_unreachable, as appropriate.

	* config/v850/v850.c (print_operand): Use gcc_assert and
	gcc_unreachable, as appropriate.
	(print_operand_address, v850_handle_data_area_attribute,
	v850_encode_data_area, construct_restore_jr, construct_save_jarl,
	v850_insert_attributes, construct_prepare_instruction): Likewise.
	* config/v850/v850.h (INITIAL_ELIMINATION_OFFSET): Likewise.
	* config/v850/v850.md (movsicc, *sasf_1): Likewise.

From-SVN: r99351

gcc/ChangeLog

+2005-05-07  Nathan Sidwell  <nathan@codesourcery.com>
+
+	* config/v850/v850.c (print_operand): Use gcc_assert and
+	gcc_unreachable, as appropriate.
+	(print_operand_address, v850_handle_data_area_attribute,
+	v850_encode_data_area, construct_restore_jr, construct_save_jarl,
+	v850_insert_attributes, construct_prepare_instruction): Likewise.
+	* config/v850/v850.h (INITIAL_ELIMINATION_OFFSET): Likewise.
+	* config/v850/v850.md (movsicc, *sasf_1): Likewise.
+
 2005-05-07  Kazu Hirata  <kazu@cs.umass.edu>
 
 	* tree-ssa-loop-ivcanon.c, config/i386/i386.c,

gcc/config/v850/v850.h

@@ -554,7 +554,7 @@ enum reg_class
   else if ((FROM) == ARG_POINTER_REGNUM)				\
    (OFFSET) = compute_frame_size (get_frame_size (), (long *)0);	\
   else									\
-    abort ();								\
+    gcc_unreachable ();							\
 }
 
 /* Keep the stack pointer constant throughout the function.  */

gcc/config/v850/v850.md

@@ -876,6 +876,7 @@
   "
 {
   rtx insn = get_last_insn_anywhere ();
+  rtx src;
 
   if (   (GET_CODE (operands[2]) == CONST_INT
        && GET_CODE (operands[3]) == CONST_INT))
@@ -903,28 +904,28 @@
       if (GET_CODE (operands[3]) != REG)
 	operands[3] = copy_to_mode_reg (SImode, operands[3]);
     }
-  if (GET_CODE (insn) == INSN
+  gcc_assert (GET_CODE (insn) == INSN
 	      && GET_CODE (PATTERN (insn)) == SET
-      && SET_DEST (PATTERN (insn)) == cc0_rtx)
-    {
-      rtx src = SET_SRC (PATTERN (insn));
+	      && SET_DEST (PATTERN (insn)) == cc0_rtx);
+    
+  src = SET_SRC (PATTERN (insn));
 
-      if (GET_CODE (src) == COMPARE)
+  switch (GET_CODE (src))
     {
+    case COMPARE:
       operands[4] = XEXP (src, 0);
       operands[5] = XEXP (src, 1);
-	}
-      else if (GET_CODE (src) == REG
-               || GET_CODE (src) == SUBREG)
-	{
+      break;
+
+    case REG:
+    case SUBREG:
       operands[4] = src;
       operands[5] = const0_rtx;
+      break;
+
+    default:
+      gcc_unreachable ();
     }
-      else
-	abort ();
-    }
-  else
-    abort ();
 }")
 
 ;; ??? Clobbering the condition codes is overkill.
@@ -1002,7 +1003,7 @@
 		(ashift:SI (match_operand:SI 2 "register_operand" "")
 			   (const_int 1))))]
   "TARGET_V850E"
-  "* abort ();")
+  "* gcc_unreachable ();")
 
 (define_insn "*sasf_2"
   [(set (match_operand:SI 0 "register_operand" "=r")