[Bug record/18059] Why can't "record instruction-history" and "record function-call-history" work with "record full"?
"cvs-commit at gcc dot gnu.org via Gdb-prs" <[email protected]>
| Newsgroups | gmane.comp.gdb.bugs.discuss |
|---|---|
| Message-ID | <[email protected]/bugzilla/> |
https://sourceware.org/bugzilla/show_bug.cgi?id=18059 --- Comment #5 from Sourceware Commits <cvs-commit at gcc dot gnu.org> --- The master branch has been updated by Guinevere Larsen <[email protected]>: https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=4d0a9f412df18d3bb8c554ae115eba7dce5e745e commit 4d0a9f412df18d3bb8c554ae115eba7dce5e745e Author: Guinevere Larsen <[email protected]> Date: Mon Jun 1 16:54:09 2026 -0300 gdb/record: extract the PC to record_full_instruction This commit makes it so the PC is not saved as part of the record_full_instruction effects, but rather gets a special location. That is because a couple of commands would really benefit from it being easy to find the PC (especially ones from record-btrace that's haven't been implemented to record-full yet, such as the ones in PR record/18059), while also possibly allowing for one fewer resizing of the effect vector (and saving an entire byte in the process). This commit also refactored record_full_read_entry_from_bfd and record_full_write_entry_to_bfd, to make them methods of record_full_reg_entry and record_full_mem_entry, and also creates similar methods for record_full_entry and record_full_instruction. These could be turned into constructors in a future step of c++ification, but it felt like too much change for a single commit. Reviewed-by: Thiago Jung Bauermann <[email protected]> Reviewed-By: Christina Schimpe <[email protected]> Approved-By: Guinevere Larsen <[email protected]> -- You are receiving this mail because: You are on the CC list for the bug.