This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 2/6] DW attribute macro MACRO_AT_func and MACRO_AT_range
- From: Doug Evans <dje at google dot com>
- To: Yao Qi <yao at codesourcery dot com>
- Cc: gdb-patches <gdb-patches at sourceware dot org>
- Date: Mon, 10 Nov 2014 11:44:49 -0800
- Subject: Re: [PATCH 2/6] DW attribute macro MACRO_AT_func and MACRO_AT_range
- Authentication-results: sourceware.org; auth=none
- References: <1414195968-3333-1-git-send-email-yao at codesourcery dot com> <1414195968-3333-3-git-send-email-yao at codesourcery dot com> <21593 dot 22575 dot 941029 dot 980760 at ruffy2 dot mtv dot corp dot google dot com> <87fvdypgap dot fsf at codesourcery dot com> <CADPb22TSAFyEY1VNghhXmL3P27_o0OPPWS0jYSdAvJGS1jE7gg at mail dot gmail dot com> <87k333izfm dot fsf at codesourcery dot com>
On Sun, Nov 9, 2014 at 6:04 PM, Yao Qi <yao@codesourcery.com> wrote:
> I am inclined to delete them via "remote_file host delete" as many
> executables func_addr[pid].x are generated. Supposing dwarf assembler
> works perfectly, people have only to concentrate on their test artifacts
> instead of func_addr[pid].x.
OTOH, leaving such things around for debug purposes is important,
or at least not making it difficult to keep such things (and editing source
files to, e.g., comment out the "file delete", crosses a threshold of
annoyance for me).
I think the simplicity of implementation and assistance to debugability
of just leaving the file there outweighs the extra code and any perceived
increase in cleanliness.
Plus the more remote file operations we add the more we slow
things down. There's a *ton* of files we leave behind,
I wouldn't worry about leaving a few more behind.
If there were another reason then that might be something
to consider.
[At least with gcc there is -save-temps. We *could* have something
like that here, but I'm ok with just leaving them.]