Re: [GSOC] LTO dump tool project

[Martin Liška <mliska at suse dot cz>]

On 06/12/2018 06:22 PM, Hrishikesh Kulkarni wrote:
> Hi,
>
> I tried tweaking the make file. I am probably missing something. Please advise.

Hi.

Next time please send diff. It's easier to read it. You'll need something like:

diff --git a/gcc/lto/config-lang.in b/gcc/lto/config-lang.in
index 7cb7a7752a1..52e452af5f3 100644
--- a/gcc/lto/config-lang.in
+++ b/gcc/lto/config-lang.in
@@ -18,8 +18,8 @@
 # <http://www.gnu.org/licenses/>.
 
 language="lto"
-compilers="lto1\$(exeext)"
-stagestuff="lto1\$(exeext)"
+compilers="lto1\$(exeext) lto-dump\$(exeext)"
+stagestuff="lto1\$(exeext) lto-dump\$(exeext)"
 
 gtfiles="\$(srcdir)/lto/lto-tree.h \$(srcdir)/lto/lto-lang.c \$(srcdir)/lto/lto.c \$(srcdir)/lto/lto.h"
 
And please rename it to lto-dump.

Martin

> Regards,
>
> Hrishikesh
>
> On Mon, Jun 11, 2018 at 1:36 PM, Martin Liška <mliska@suse.cz> wrote:
> > On 06/08/2018 07:16 PM, Hrishikesh Kulkarni wrote:
> > > Hi,
> > >
> > > -fdump-lto-body=foo
> > > will dump gimple body of the function foo
> > >
> > > foo (int a, int b)
> > > {
> > >    <bb 2> [local count: 1073741825]:
> > >    _3 = a_1(D) + b_2(D);
> > >    return _3;
> > >
> > > }
> > >
> > > Please find the diff file attached herewith.
> > >
> > > Regards,
> > > Hrishikesh
> > >
> > > On Fri, Jun 8, 2018 at 7:15 PM, Martin Liška <mliska@suse.cz> wrote:
> > > > On 06/08/2018 03:40 PM, Martin Liška wrote:
> > > > > There's wrong declaration of the function in header file. I'll fix it soon
> > > > > on trunk. Please carry on with following patch:
> > > >
> > > > Fixed in r261327.
> > > >
> > > > Martin
> >
> > Hi.
> >
> > Thanks for another new option. I have some comments:
> >
> > 1) remove these files from git branch:
> > diff --git a/gcc/cscope.out b/gcc/cscope.out
> > diff --git a/gcc/err b/gcc/err
> >
> > 2) You have repeated pattern:
> >
> > FOR_EACH_SYMBOL (node)
> >    if (!strcmp (flag_lto_dump_symbol, node->name ()))
> >       some_action();
> >
> > Let's come up with function symtab->find_symbol_by_name.
> >
> > Martin