gdb: make symbols const in struct inline_state

Make the inline_state::skipped_symbols a vector of 'const symbol *',
adding the const qualifier.

There's only a couple of places this leaks into the rest of GDB and in
both places its fine for the symbol to become const.

There should be no functional change after this commit.

Approved-By: Simon Marchi <simon.marchi@efficios.com>
This commit is contained in:
Andrew Burgess 2024-08-13 13:07:19 +01:00
parent a6feefa6eb
commit 3e9907d849
4 changed files with 7 additions and 7 deletions

View File

@ -2859,7 +2859,7 @@ find_frame_sal (const frame_info_ptr &frame)
if (frame_inlined_callees (frame) > 0)
{
struct symbol *sym;
const symbol *sym;
/* If the current frame has some inlined callees, and we have a next
frame, then that frame must be an inlined frame. In this case

View File

@ -992,7 +992,7 @@ prepare_one_step (thread_info *tp, struct step_command_fsm *sm)
Use inlined_subroutine info to make the range more narrow. */
if (inline_skipped_frames (tp) > 0)
{
symbol *sym = inline_skipped_symbol (tp);
const symbol *sym = inline_skipped_symbol (tp);
if (sym->aclass () == LOC_BLOCK)
{
const block *block = sym->value_block ();

View File

@ -36,7 +36,7 @@
struct inline_state
{
inline_state (thread_info *thread_, int skipped_frames_, CORE_ADDR saved_pc_,
std::vector<symbol *> &&skipped_symbols_)
std::vector<const symbol *> &&skipped_symbols_)
: thread (thread_), skipped_frames (skipped_frames_), saved_pc (saved_pc_),
skipped_symbols (std::move (skipped_symbols_))
{}
@ -58,7 +58,7 @@ struct inline_state
/* Only valid if SKIPPED_FRAMES is non-zero. This is the list of all
function symbols that have been skipped, from inner most to outer
most. It is used to find the call site of the current frame. */
std::vector<struct symbol *> skipped_symbols;
std::vector<const symbol *> skipped_symbols;
};
static std::vector<inline_state> inline_states;
@ -342,7 +342,7 @@ void
skip_inline_frames (thread_info *thread, bpstat *stop_chain)
{
const struct block *frame_block, *cur_block;
std::vector<struct symbol *> skipped_syms;
std::vector<const symbol *> skipped_syms;
int skip_count = 0;
/* This function is called right after reinitializing the frame
@ -419,7 +419,7 @@ inline_skipped_frames (thread_info *thread)
/* If one or more inlined functions are hidden, return the symbol for
the function inlined into the current frame. */
struct symbol *
const symbol *
inline_skipped_symbol (thread_info *thread)
{
inline_state *state = find_inline_frame_state (thread);

View File

@ -62,7 +62,7 @@ int inline_skipped_frames (thread_info *thread);
/* If one or more inlined functions are hidden, return the symbol for
the function inlined into the current frame. */
struct symbol *inline_skipped_symbol (thread_info *thread);
const symbol *inline_skipped_symbol (thread_info *thread);
/* Return the number of functions inlined into THIS_FRAME. Some of
the callees may not have associated frames (see