<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 17, 2013 at 4:59 PM, Peter Wang <span dir="ltr"><<a href="mailto:novalazy@gmail.com" target="_blank">novalazy@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, 17 Sep 2013 16:49:15 +1000, Julien Fischer <<a href="mailto:jfischer@opturion.com">jfischer@opturion.com</a>> wrote:<br>

> On Tue, Sep 17, 2013 at 4:38 PM, Peter Wang <<a href="mailto:novalazy@gmail.com">novalazy@gmail.com</a>> wrote:<br>
><br>
> > On Tue, 17 Sep 2013 02:30:43 +1000 (EST), Julien Fischer <<br>
> > <a href="mailto:jfischer@opturion.com">jfischer@opturion.com</a>> wrote:<br>
> > ><br>
> > > For review by anyone.<br>
> > ><br>
> > > -----------<br>
> > ><br>
> > > Compile time evaluation of string.count_codepoints.<br>
> > ><br>
> > > compiler/const_prop.m:<br>
> > >      Do compile time evaluation of string.count_codepoints where<br>
> > possible.<br>
> > ><br>
> > >      Fix an old error in a comment: functions with two arguments are<br>
> > >      binary, not unary.<br>
> > ><br>
> > > Julien.<br>
> ><br>
> > I'm curious about the motivation for this.<br>
> ><br>
><br>
> Which one, fixing the incorrect comment or doing compile time evaluation of<br>
> count_codepoints?  ;-)<br>
><br>
> There's no really strong motivation other than that I noticed that some<br>
> code that I was working on<br>
> G12 would be smaller if we did the above substitution (because it enables<br>
> further simplifications<br>
> to take place).<br>
<br>
</div></div>I don't like how brittle the recognition of known procedures is,<br>
so I suggest not adding to it without good reason.<br></blockquote><div><br></div><div style>It isn't all that brittle is it?  I don't remember it being particularly troublesome in the past.</div><div style><br></div>
<div style>Cheers,</div><div style>Julien.</div><div><br></div><div><br></div><div> </div></div></div></div>