Looks ok, although with the changeset you pushed, there is no desugaring
so you should probably update the bug description to reflect that.
(I probably would have kept the desugaring in the SM case, but probably
not a big deal either way)
--Sean
On 5/22/19 7:06 AM, Claes Redestad wrote:
True, I'll skip the desugaring of the else branch, keeps it cleaner.
On 2019-05-22 12:50, Alan Bateman wrote:
On 22/05/2019 11:34, Claes Redestad wrote:
:
You mean the SM case? Not as much as the no-SM case, no, which can
be helped a lot in the smallest affected tests.
Yes, I don't think the SM case should be performance/startup concern
here. Up to you, I don't think it matters but I'm sure someone will be
refactor it again.
-Alan
