<div dir="ltr">Looks good, but since it is an optimization<div style>we will merge it to master (after testing it</div><div style>in our daily builds a few days).</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Tue, Apr 9, 2013 at 11:40 PM, Anthony Ramine <span dir="ltr"><<a href="mailto:n.oxyde@gmail.com" target="_blank">n.oxyde@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
The Core Erlang compiler simplifies let expressions by moving them into their argument when it is another let or a case expression. It can then perform other optimizations such as removing the let expressions altogether, sometimes saving BEAM registers later in the process.<br>

<br>
This patch teaches sys_core_fold how to move let expressions into sequence arguments.<br>
<br>
        git fetch <a href="https://github.com/nox/otp.git" target="_blank">https://github.com/nox/otp.git</a> move-let-into-seq<br>
<br>
        <a href="https://github.com/nox/otp/compare/erlang:maint...move-let-into-seq" target="_blank">https://github.com/nox/otp/compare/erlang:maint...move-let-into-seq</a><br>
        <a href="https://github.com/nox/otp/compare/erlang:maint...move-let-into-seq.patch" target="_blank">https://github.com/nox/otp/compare/erlang:maint...move-let-into-seq.patch</a><br>
<br>
For the record, I managed to encounter this while working on the file optimization in branch file-receive-optim. You can find a short file showing seq in let in the following gist:<br>
<br>
        <a href="https://gist.github.com/nox/2e33fe9a85e035caadda#file-t-core-L14" target="_blank">https://gist.github.com/nox/2e33fe9a85e035caadda#file-t-core-L14</a><br>
        <a href="https://gist.github.com/nox/2e33fe9a85e035caadda#file-t-erl" target="_blank">https://gist.github.com/nox/2e33fe9a85e035caadda#file-t-erl</a><br>
<br>
Regards,<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Anthony Ramine<br>
<br>
_______________________________________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>Björn Gustavsson, Erlang/OTP, Ericsson AB
</div>