https://bugs.kde.org/show_bug.cgi?id=211031

--- Comment #25 from Jefferson Carpenter <jeffersoncarpent...@gmail.com> ---
Removed the spurious output.  Did not manually add a mapping, to fix the
"DiCfsi invariant (1) verification failed" message.  Also removed the dwarf3
code - the patch still seems to work for me, and native PE files will of course
not have dwarf3 debug info.

There are still some things I'm very unsure about whether this patch does
correctly.

1: Latest version, I changed the search_all_symtabs logic to search for text
sections using di->text_present and so on, instead of looking at di->fsm
properties.  It seems logical, but since I'm not very familiar with the
Valgrind code it could be a breaking change.  Someone would have to look at it.

2: The way read_pe_debug_info is called into is IMO a messy hack.  I did add a
function for detecting whether a binary is a PE object file, but I think
passing a boolean argument into di_notify_ACHIEVE_ACCEPT_STATE is not the
cleanest way to do it.  The way PDB debug info is requested is through client
requests, so maybe PE debug info should be requested the same way.

3: There's generally a little bit of code duplication.  Some duplicate "if
(!ML_(img_valid)(...))" statements I could try to combine or replace with a
macro.  Also the PE executable file format is modeled both in readpdb.c and
readpe.c currently and should be factored out into a header.  I just didn't
bother to do that at this early stage.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to