[m-rev.] for review: call sequence numbers in the deep profiler

[Julien Fischer]

On Wed, 4 Oct 2006, Zoltan Somogyi wrote:

> Enhance the deep profiler by including in the output a measure of execution
> time that has much finer granularity than clock ticks: a count of procedure
> calls. The deep profiler does not yet display this data; that is for a later
> diff.
>
> runtime/mercury_deep_profiling.[ch]:
> 	Add a field containing this counter if the appropriate macro is
> 	defined. Initialize the new field when the profiling metrics structure
> 	it is in is initialized.
>
> 	Use masks that fit into one byte for the commonly needed fields
> 	of the profiling metrics structure.
>
> runtime/mercury_conf_params.h:
> 	Define this macro in deep profiling grades unless its absence is
> 	expressly requested (for performance tests of the deep profiler
> 	itself).
>
> runtime/mercury_grade.h:
> 	Increment the binary compatibility number for the deep profiling
> 	grades.
>
> runtime/mercury_deep_{call,redo,leave}_port_body.h:
> 	Update the call sequence number global at call sites.
>
> 	Update the new field when control enters a procedure invocation
> 	through the call or redo ports, and when it leaves the invocation
> 	through the exit or fail ports.
>
> library/exception.m:
> 	Update the new field when control leaves a procedure invocation
> 	through the excp port.
>
> deep_profiler/measurements.m:
> 	Include the new field in the deep profiler's version of the
> 	profiling metric structure.
>
> deep_profiler/read_profile.m:
> 	Read in the updated format now generated by the runtime.
>
> deep_profiler/dump.m:
> 	Include the new field in any dumps of the deep profiler's
> 	internal data structures.

That looks okay.

Julien.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------