This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Add call site line number with -fdump-ipa-inline-all switch


> > > When a function is calling another function multiple times at
> different
> > > locations all of which may not be inlined, it is difficult to figure
> out
> > > which of the calls where inlined without the line number information
> of
> > > the call site.
> > > This patch adds the line number of the call site in the dump_file.
> > >
> > > Tested and Bootstrapped on x86_64.
> > > Okay for the trunk?
> > 
> > This seems fine in general, but you probably want to print just
> standard
> > file.c:1234 type locator, right?  Especially in LTO context.
> > 
> 
> Yes. Attached patch implements this. Tested and Bootstrapped on x86_64.
> Is this OK for trunk?

OK
Honza
> 
> Thanks,
> Dwarak



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]