mirror of
https://sourceware.org/git/binutils-gdb.git
synced 2025-01-18 12:24:38 +08:00
* gdb_assert.h (gdb_assert_not_reached): New macro.
(gdb_assert_fail): Fix typo in comment. * avr-tdep.c (avr_return_value): Use gdb_assert_not_reached instead of gdb_assert (0). * darwin-nat.c (darwin_check_new_threads): Ditto. * dwarf2read.c (dwarf2_get_section_info): Ditto. (munmap_section_buffer): Ditto. * m32c-tdep.c (make_types): Ditto. (m32c_decode_srcdest4, m32c_decode_sd23, m32c_frame_base): Ditto. * macroexp.c (get_character_constant): Ditto. (get_string_literal): Ditto. * mep-tdep.c (mep_pseudo_cr_size): Ditto. (mep_pseudo_cr_index, mep_register_type): Ditto. (mep_pseudo_register_read, mep_pseudo_register_write): Ditto. (mep_get_insn, mep_analyze_prologue): Ditto. * objfiles.c (qsort_cmp): Ditto. * prologue-value.c (pv_is_identical): Ditto. * record.c (record_get_loc): Ditto. * value.c (value_static_field): Ditto. * xtensa-tdep.c (call0_track_op): Ditto.
This commit is contained in:
parent
c92817ce8d
commit
f3574227a8
@ -1,3 +1,26 @@
|
||||
2010-08-18 Doug Evans <dje@google.com>
|
||||
|
||||
* gdb_assert.h (gdb_assert_not_reached): New macro.
|
||||
(gdb_assert_fail): Fix typo in comment.
|
||||
* avr-tdep.c (avr_return_value): Use gdb_assert_not_reached instead of
|
||||
gdb_assert (0).
|
||||
* darwin-nat.c (darwin_check_new_threads): Ditto.
|
||||
* dwarf2read.c (dwarf2_get_section_info): Ditto.
|
||||
(munmap_section_buffer): Ditto.
|
||||
* m32c-tdep.c (make_types): Ditto.
|
||||
(m32c_decode_srcdest4, m32c_decode_sd23, m32c_frame_base): Ditto.
|
||||
* macroexp.c (get_character_constant): Ditto.
|
||||
(get_string_literal): Ditto.
|
||||
* mep-tdep.c (mep_pseudo_cr_size): Ditto.
|
||||
(mep_pseudo_cr_index, mep_register_type): Ditto.
|
||||
(mep_pseudo_register_read, mep_pseudo_register_write): Ditto.
|
||||
(mep_get_insn, mep_analyze_prologue): Ditto.
|
||||
* objfiles.c (qsort_cmp): Ditto.
|
||||
* prologue-value.c (pv_is_identical): Ditto.
|
||||
* record.c (record_get_loc): Ditto.
|
||||
* value.c (value_static_field): Ditto.
|
||||
* xtensa-tdep.c (call0_track_op): Ditto.
|
||||
|
||||
2010-08-18 Tom Tromey <tromey@redhat.com>
|
||||
|
||||
PR symtab/11919:
|
||||
|
@ -922,7 +922,7 @@ avr_return_value (struct gdbarch *gdbarch, struct type *func_type,
|
||||
else if (TYPE_LENGTH (valtype) <= 8)
|
||||
lsb_reg = 18;
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected type length");
|
||||
|
||||
if (writebuf != NULL)
|
||||
{
|
||||
|
@ -363,7 +363,7 @@ darwin_check_new_threads (struct inferior *inf)
|
||||
old_ix++;
|
||||
continue;
|
||||
}
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected thread case");
|
||||
}
|
||||
|
||||
if (darwin_inf->threads)
|
||||
|
@ -1585,7 +1585,7 @@ dwarf2_get_section_info (struct objfile *objfile, const char *section_name,
|
||||
else if (section_is_p (section_name, FRAME_SECTION))
|
||||
info = &data->frame;
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected section");
|
||||
|
||||
if (info->asection != NULL && info->size != 0 && info->buffer == NULL)
|
||||
/* We haven't read this section in yet. Do it now. */
|
||||
@ -14263,7 +14263,7 @@ munmap_section_buffer (struct dwarf2_section_info *info)
|
||||
gdb_assert (munmap ((void *) map_begin, map_length) == 0);
|
||||
#else
|
||||
/* Without HAVE_MMAP, we should never be here to begin with. */
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("no mmap support");
|
||||
#endif
|
||||
}
|
||||
}
|
||||
|
@ -42,7 +42,7 @@
|
||||
#endif
|
||||
#endif
|
||||
|
||||
/* This prints an "Assertion failed" message, aksing the user if they
|
||||
/* This prints an "Assertion failed" message, asking the user if they
|
||||
want to continue, dump core, or just exit. */
|
||||
#if defined (ASSERT_FUNCTION)
|
||||
#define gdb_assert_fail(assertion, file, line, function) \
|
||||
@ -54,4 +54,15 @@
|
||||
assertion)
|
||||
#endif
|
||||
|
||||
/* The canonical form of gdb_assert (0).
|
||||
MESSAGE is a string to include in the error message. */
|
||||
|
||||
#if defined (ASSERT_FUNCTION)
|
||||
#define gdb_assert_not_reached(message) \
|
||||
internal_error (__FILE__, __LINE__, "%s: %s", ASSERT_FUNCTION, _(message))
|
||||
#else
|
||||
#define gdb_assert_not_reached(message) \
|
||||
internal_error (__FILE__, __LINE__, _(message))
|
||||
#endif
|
||||
|
||||
#endif /* gdb_assert.h */
|
||||
|
@ -187,7 +187,7 @@ make_types (struct gdbarch *arch)
|
||||
break;
|
||||
|
||||
default:
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected mach");
|
||||
}
|
||||
|
||||
/* The builtin_type_mumble variables are sometimes uninitialized when
|
||||
@ -1242,7 +1242,7 @@ m32c_decode_srcdest4 (struct m32c_pv_state *st,
|
||||
case 0xf: sd.addr = pv_constant (m32c_udisp16 (st)); break;
|
||||
|
||||
default:
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected srcdest4");
|
||||
}
|
||||
|
||||
return sd;
|
||||
@ -1301,7 +1301,7 @@ m32c_decode_sd23 (struct m32c_pv_state *st, int code, int size, int ind)
|
||||
case 0x0f: sd.addr = pv_constant (m32c_udisp16 (st)); break;
|
||||
case 0x0e: sd.addr = pv_constant (m32c_udisp24 (st)); break;
|
||||
default:
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected sd23");
|
||||
}
|
||||
|
||||
if (ind)
|
||||
@ -1886,7 +1886,7 @@ m32c_frame_base (struct frame_info *this_frame,
|
||||
return 0;
|
||||
|
||||
default:
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected prologue kind");
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -334,7 +334,7 @@ get_character_constant (struct macro_buffer *tok, char *p, char *end)
|
||||
else if (*p == 'L' || *p == 'u' || *p == 'U')
|
||||
p += 2;
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected character constant");
|
||||
|
||||
body_start = p;
|
||||
for (;;)
|
||||
@ -389,7 +389,7 @@ get_string_literal (struct macro_buffer *tok, char *p, char *end)
|
||||
else if (*p == 'L' || *p == 'u' || *p == 'U')
|
||||
p += 2;
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected string literal");
|
||||
|
||||
for (;;)
|
||||
{
|
||||
|
@ -804,7 +804,7 @@ mep_pseudo_cr_size (int pseudo)
|
||||
|| IS_FP_CR64_REGNUM (pseudo))
|
||||
return 64;
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected coprocessor pseudo register");
|
||||
}
|
||||
|
||||
|
||||
@ -833,7 +833,7 @@ mep_pseudo_cr_index (int pseudo)
|
||||
else if (IS_FP_CR64_REGNUM (pseudo))
|
||||
return pseudo - MEP_FIRST_FP_CR64_REGNUM;
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected coprocessor pseudo register");
|
||||
}
|
||||
|
||||
|
||||
@ -1109,7 +1109,7 @@ mep_register_type (struct gdbarch *gdbarch, int reg_nr)
|
||||
return builtin_type (gdbarch)->builtin_uint64;
|
||||
}
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected cr size");
|
||||
}
|
||||
|
||||
/* All other registers are 32 bits long. */
|
||||
@ -1180,7 +1180,7 @@ mep_pseudo_register_read (struct gdbarch *gdbarch,
|
||||
|| IS_FP_CR64_REGNUM (cookednum))
|
||||
mep_pseudo_cr64_read (gdbarch, regcache, cookednum, buf);
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected pseudo register");
|
||||
}
|
||||
|
||||
|
||||
@ -1263,7 +1263,7 @@ mep_pseudo_register_write (struct gdbarch *gdbarch,
|
||||
else if (IS_CCR_REGNUM (cookednum))
|
||||
regcache_raw_write (regcache, mep_pseudo_to_raw[cookednum], buf);
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected pseudo register");
|
||||
}
|
||||
|
||||
|
||||
@ -1488,7 +1488,7 @@ mep_get_insn (struct gdbarch *gdbarch, CORE_ADDR pc, long *insn)
|
||||
|
||||
/* We'd better be in either core, 32-bit VLIW, or 64-bit VLIW mode. */
|
||||
else
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected vliw mode");
|
||||
}
|
||||
|
||||
/* Otherwise, the top two bits of the major opcode are (again) what
|
||||
@ -1792,8 +1792,7 @@ mep_analyze_prologue (struct gdbarch *gdbarch,
|
||||
int disp = SWBH_32_OFFSET (insn);
|
||||
int size = (IS_SB (insn) ? 1
|
||||
: IS_SH (insn) ? 2
|
||||
: IS_SW (insn) ? 4
|
||||
: (gdb_assert (0), 1));
|
||||
: (gdb_assert (IS_SW (insn)), 4));
|
||||
pv_t addr = pv_add_constant (reg[rm], disp);
|
||||
|
||||
if (pv_area_store_would_trash (stack, addr))
|
||||
|
@ -1068,7 +1068,7 @@ qsort_cmp (const void *a, const void *b)
|
||||
return 1;
|
||||
|
||||
/* We should have found one of the sections before getting here. */
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("section not found");
|
||||
}
|
||||
else
|
||||
{
|
||||
@ -1083,12 +1083,12 @@ qsort_cmp (const void *a, const void *b)
|
||||
return 1;
|
||||
|
||||
/* We should have found one of the objfiles before getting here. */
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("objfile not found");
|
||||
}
|
||||
}
|
||||
|
||||
/* Unreachable. */
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected code path");
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
@ -203,7 +203,7 @@ pv_is_identical (pv_t a, pv_t b)
|
||||
case pvk_register:
|
||||
return (a.reg == b.reg && a.k == b.k);
|
||||
default:
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected prologue value kind");
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -454,7 +454,7 @@ record_get_loc (struct record_entry *rec)
|
||||
return rec->u.reg.u.buf;
|
||||
case record_end:
|
||||
default:
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected record_entry type");
|
||||
return NULL;
|
||||
}
|
||||
}
|
||||
|
@ -1917,7 +1917,7 @@ value_static_field (struct type *type, int fieldno)
|
||||
break;
|
||||
}
|
||||
default:
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected field location kind");
|
||||
}
|
||||
|
||||
return retval;
|
||||
|
@ -2057,7 +2057,7 @@ call0_track_op (struct gdbarch *gdbarch,
|
||||
}
|
||||
break;
|
||||
default:
|
||||
gdb_assert (0);
|
||||
gdb_assert_not_reached ("unexpected instruction kind");
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user