[m-rev.] for post-commit review: mdprof and >2^15 vars

Julien Fischer juliensf at csse.unimelb.edu.au
Mon Oct 11 15:03:14 AEDT 2010

On Mon, 11 Oct 2010, Zoltan Somogyi wrote:

> Allow more than 2^15 vars in a procedure representation.
> mdbcomp/program_representation.m:
> 	Allow a variable number to be represented by four bytes as well as
> 	two and one. This means that we also have to represent the number
> 	of variables in a procedure using a four-byte number, not a two-byte
> 	number.
> 	Use four bytes to represent line numbers. Programs that overflow
> 	16-bit var numbers may also overflow 16 bit line numbers.
> 	These requires a change in the deep profiler data's binary
> 	compatibility version number.

It would be worth mentioning that the deep profiler now supports a
larger number of varaibles in a procedure in the NEWS file.

> compiler/prog_rep.m:
> 	Encode vars using four bytes if necessary. Be consistent in using
> 	only signed 8-bit as well as signed 16-bit numbers.
> compiler/implicit_parallelism.m:
> 	Conform to the change in program_representation.m.
> deep_profiler/profile.m:
> deep_profiler/read_profile.m:
> 	Add a compression flag to the set of flags read from the data file.
> 	Put the flags into the profile_stats as a group, not one-by-one.
> deep_profiler/canonical.m:
> deep_profiler/create_report.m:
> deep_profiler/dump.m:
> deep_profiler/mdprof_feedback.m:
> deep_profiler/old_html_format.m:
> deep_profiler/old_query.m:

Is the code in the old_*.m files still necessary?

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

More information about the reviews mailing list