[m-rev.] for post-commit review: fix a bug with require_switch_arms_detism

Julien Fischer jfischer at opturion.com
Mon Apr 11 10:19:06 AEST 2016


On Sun, 10 Apr 2016, Zoltan Somogyi wrote:

> For review by anyone.
>
> Zoltan.
>
> Fix a bug in handling require_switch_arms_detism.
> 
> compiler/cse_detection.m:
>     Don't pull unifications out of the arms of switches inside
>     require_switch_arms_detism scopes, since this can affect make
>     the scope ineffective, leading to error messages that should be
>     generated not being generated.
> 
> compiler/hlds_goal.m:
>     Fix some misleading variable names.
> 
> tests/invalid/require_switch_arms_detism.{m,err_exp}:
>     A regression test for the bug.
> 
> tests/invalid/Mmakefile:
>     Enable the new test.

That looks fine.

Julien.


More information about the reviews mailing list