[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