[m-rev.] for review: fix confusing call sites in deep profiles
Julien Fischer
jfischer at opturion.com
Thu Mar 20 13:25:57 AEDT 2025
On Thu, 20 Mar 2025 at 11:56, Zoltan Somogyi <zoltan.somogyi at runbox.com> wrote:
> Include file names in call site static structures.
>
> Fix a problem that arises in the deep profiler if the program being profiled
> was using both intermodule optimization and inlining. The issue was that
> even though runtime/mercury_deep_profiling.c had access, for every call site
> to the full context of that call site, containing both the file name and
> the line number, it wrote out *only* the line number. The deep profiler
> then got the file name from the file name stored in the proc_static structure
> of the procedure containing the call site.
That's fine.
Julien.
More information about the reviews
mailing list