Summary: | readelf erases CIE initial register state | ||
---|---|---|---|
Product: | binutils | Reporter: | Alan Modra <amodra> |
Component: | binutils | Assignee: | Alan Modra <amodra> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | sevaa |
Priority: | P2 | ||
Version: | 2.39 | ||
Target Milestone: | 2.39 | ||
Host: | Target: | ||
Build: | Last reconfirmed: |
Description
Alan Modra
2022-06-15 12:56:47 UTC
I agree that changing the test from >= 0 to > 0 will result in proper readelf behaviour, but I'm inclined to think that there is no need to set col_type at all in the first pass. (In reply to Alan Modra from comment #1) > I agree that changing the test from >= 0 to > 0 will result in proper > readelf behaviour Hmm, no it won't. That might lose reg names from the header, or entries on lines of the --debug-dump=frames-interp display. The master branch has been updated by Alan Modra <amodra@sourceware.org>: https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=45bf072b34068b5a98947862b2aa183ab646e7ea commit 45bf072b34068b5a98947862b2aa183ab646e7ea Author: Alan Modra <amodra@gmail.com> Date: Wed Jun 15 22:30:51 2022 +0930 PR29250, readelf erases CIE initial register state PR 29250 binutils/ * dwarf.c (display_debug_frames): Set col_type[reg] on sizing pass over FDE to cie->col_type[reg] if CIE specifies reg. Handle DW_CFA_restore and DW_CFA_restore_extended on second pass using the same logic. Remove unnecessary casts. Don't call frame_need_space on second pass over FDE. gas/ * testsuite/gas/i386/ehinterp.d, * testsuite/gas/i386/ehinterp.s: New test. * testsuite/gas/i386/i386.exp: Run it. Fixed for 2.39 |