[m-rev.] for review: fix deep profiling runtime compilation failure

Julien Fischer juliensf at csse.unimelb.edu.au
Fri Feb 16 15:23:11 AEDT 2007


On Fri, 16 Feb 2007, Simon Taylor wrote:

>
> On 16-Feb-2007, Julien Fischer <juliensf at csse.unimelb.edu.au> wrote:
>> This change breaks the deep profiling grades (and possibly also the
>> debugging ones).  There is a fundamental problem with the way the
>> bitmap type is defined.  It's a foreign type but (parts of) the runtime also
>> treat it as though it were a builtin type.
>>
>> This is a problem because the runtime is now referring to entities that
>> are defined in the library.  The solution here is to make it either one
>> or the other.
>
> This is the same as for arrays (they are defined in exactly the same way).

Ah, I didn't realise that was the case.

> The short-term fix is to abort if compare_representation/3 is called for
> bitmaps as well.

This is fairly yucky.  We should consider allowing 
compare_representation/3 to be defined for foreign_types.

> Ralph Becket wrote:
>> Why does the compiler/runtime need to know about bitmaps at all?
>
> For deep copying and tabling.

and deep copying for foreign types as well.  (I'm not sure about tabling.)

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
--------------------------------------------------------------------------



More information about the reviews mailing list