[m-dev.] for review: the second batch of trace ports

[Fergus Henderson]

Zoltan Somogyi, you wrote:
> 
> compiler/hlds_goal.m:
> 	Add an extra slot to each hlds_goal_info structure. This slot
> 	holds information about the position of the goal within the
> 	procedure, in the form of a description of the path from the
> 	root of the main goal to this goal.

I'm a bit concerned about the efficiency impact of this.

Couldn't you store it in the code_info rather than the goal_info?

> New File: compiler/goal_path.m
> 
> % This module looks after goal paths, which associate each goal
> % with its position in a procedure definition,

This comment is incomplete.

(Also what does "looks after" mean?  You should be more precise.)

-- 
Fergus Henderson <fjh at cs.mu.oz.au>   |  "I have always known that the pursuit
WWW: <http://www.cs.mu.oz.au/~fjh>   |  of excellence is a lethal habit"
PGP: finger fjh at 128.250.37.3         |     -- the last words of T. S. Garp.