Re: [dwp patch] Add --exec option.

[Ian Lance Taylor <iant at google dot com>]

On Fri, Jan 25, 2013 at 5:16 PM, Cary Coutant <ccoutant@google.com> wrote:
> This patch adds a -e/--exec option to dwp, to fetch the list of .dwo
> files from the (linked) executable's debug info. With this option, the
> .dwo files don't need to be listed explicitly, and the output file
> name defaults to "<exe>.dwp".
>
> Tested on x86_64. OK?
>
> -cary
>
>
> gold/
>         * dwp.cc (File_list): New typedef.
>         (Dwo_name_info_reader): New class.
>         (Dwo_id_info_reader::Dwo_id_info_reader): Remove unused parameters.
>         (Dwo_id_info_reader::visit_top_die): Remove unused member function.
>         (Dwo_file::~Dwo_file): Delete input_file_ after obj_.
>         (Dwo_file::read_executable): New function.
>         (Dwo_file::read): Move common setup code to ...
>         (Dwo_file::make_object): ... here.
>         (dwp_options): Add --exec/-e.
>         (usage): Likewise.
>         (main): Likewise.

This is OK.

Thanks.

Ian