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?

Thanks,
Dwarak

Attachment: inline-lndump-02.patch
Description: inline-lndump-02.patch


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