[m-rev.] for review: bug fix for asm_fast.gc.rt

Zoltan Somogyi zs at cs.mu.OZ.AU
Thu Feb 21 16:13:06 AEDT 2002


On 20-Feb-2002, Peter Ross <peter.ross at miscrit.be> wrote:
> what was the problem, when?

I sent the wrong copy of the Log file. The right one says:

compiler/dupelim.m:
	Fix a bug that caused a compiler abort in .rt grades when compiling the
	compiler-generated comparison procedure for the type goal_path_step
	in browser/program_representation.m. The problem was that when trying
	to find a generalization of two instruction sequences, we handled the
	case of one instruction sequence starting with a comment that the other
	instruction did not have only when the other instruction sequence
	wasn't empty. With high values of the --compare-specialization option,
	previous invocations of dupelim could create situations in which this
	oversight lead to a compiler abort.


> I haven't looked at the rest of the diff, yet.

But Simon has, and he says its OK, so I will commit it now.

Zoltan.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list