<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 1, 2013 at 10:39 PM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im"><br>
On Oct 1, 2013, at 10:20 AM, Nick Lewycky <<a href="mailto:nlewycky@gmail.com">nlewycky@gmail.com</a>> wrote:<br>
<br>
><br>
>  The optimization was "important" for some SPEC test, but I think we're past caring about that.<br>
<br>
</div>Not necessarily.  Has anyone measured the impact on SPEC and other benchmarks?<br></blockquote><div><br></div><div>I'm going to submit this anyway. See Reid's comment in <a href="http://llvm-reviews.chandlerc.com/D1754">http://llvm-reviews.chandlerc.com/D1754</a> - looks like this optimization has incorrect assumptions about C programs.</div>
<div><div>$ cat tmp/glob_di/main.c</div><div>static int glob;</div><div>static int rec;</div><div><br></div><div>int main() {</div><div>  glob++;</div><div>  if (!rec) {</div><div>    rec = 1;</div><div>    main();</div><div>
  }</div><div>  return glob;</div><div>}</div><div>$ gcc -O2 tmp/glob_di/main.c ; ./a.out ; echo $?</div><div>2</div><div>$ ./bin/clang -O2 tmp/glob_di/main.c ; ./a.out ; echo $?</div><div>1</div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div class="im"><br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D1769" target="_blank">http://llvm-reviews.chandlerc.com/D1769</a><br>
><br>
> BRANCH<br>
>  svn<br>
><br>
> ARCANIST PROJECT<br>
>  llvm<br>
</div>> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div></div>