[m-rev.] for review: structured MLDS var names

Julien Fischer jfischer at opturion.com
Mon Apr 24 14:54:48 AEST 2017


Hi Zoltan,

On Mon, 24 Apr 2017, Zoltan Somogyi wrote:

>
>
> On Sun, 23 Apr 2017 22:31:02 +1000 (AEST), Julien Fischer <jfischer at opturion.com> wrote:
>>>     This design has a couple of minor problems. The first is that there is
>>>     no place in the compiler where all the variable names are visible at once,
>>>     and without such a place, we cannot be sure that two names constructed
>>>     for different purposes don't accidentally end up with the same name.
>>>     The probability of such a clash used to be astronomically small
>>
>> See bug #92 <https://bugs.mercurylang.org/view.php?id=92>.
>
> I have tried to reproduce that problem with my installed compiler
> (which does not have this diff), but failed.

That's not surprising, IIRC I couldn't reproduce it with the rotd
from a few weeks after I found it.

> I know it was a very long time ago, but do you remember anything about
> the context of the duplication? In other words, what did the generated
> code use the nested occurrence of HeadVar__1_1 for?

Sorry, I've long since forgotten the details, and that code has long
since been eliminated from the program (the Zinc compiler) that it was a
part of.

Julien.


More information about the reviews mailing list