Bug 31363 - Remove support for .gdb_index
Summary: Remove support for .gdb_index
Status: NEW
Alias: None
Product: gdb
Classification: Unclassified
Component: symtab (show other bugs)
Version: HEAD
: P2 normal
Target Milestone: ---
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on: 23504
Blocks:
  Show dependency tree / graph
 
Reported: 2024-02-09 20:03 UTC by Tom Tromey
Modified: 2024-02-10 21:48 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:
Last reconfirmed:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Tromey 2024-02-09 20:03:48 UTC 
Now that the .debug_names writer has been rewritten,
I think we should plan to remove .gdb_index.

While .debug_names has some drawbacks (reuse of the
string table means more data to download when just
the index is desired), it is also better documented
and integrates better with the new DWARF scanner.
Comment 1 Frank Ch. Eigler 2024-02-10 21:04:26 UTC 
What about preexisting binaries that have .gdb_index built into them but no .debug_names?
Comment 2 Tom Tromey 2024-02-10 21:48:49 UTC 
(In reply to Frank Ch. Eigler from comment #1)
> What about preexisting binaries that have .gdb_index built into them but no
> .debug_names?

We don't have to delete the reader immediately.

Worth noting here that .gdb_index won't easily work with lazy CU
expansion, if we end up implementing that.