<div dir="ltr">On Thu, Jun 20, 2013 at 12:13 PM, Argyrios Kyrtzidis <span dir="ltr"><<a href="mailto:akyrtzi@gmail.com" target="_blank" class="cremed">akyrtzi@gmail.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div class="h5"><div><div>On Jun 20, 2013, at 12:40 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com" target="_blank" class="cremed">chandlerc@google.com</a>> wrote:</div>
<br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 25, 2012 at 1:38 PM, Argyrios Kyrtzidis <span dir="ltr"><<a href="mailto:akyrtzi@gmail.com" target="_blank" class="cremed">akyrtzi@gmail.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 style="overflow:hidden">Author: akirtzidis<br>
Date: Sat Feb 25 15:38:16 2012<br>
New Revision: 151460<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=151460&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=151460&view=rev</a><br>
Log:<br>
When evaluating integer expressions handle <span>logical</span> operators outside<br>
VisitBinaryOperator() to reduce stack pressure for source with huge number<br>
of <span>logical</span> operators.<br>
<br>
Fixes <a class="cremed">rdar://10913206</a>.<br>
<br>
Added:<br>
    cfe/trunk/test/Sema/<span>many</span>-<span>logical</span>-<span>ops</span>.<span>c</span></div></blockquote></div><br>Argiris, this is the 4th slowest test at almost 20 seconds. It makes me wonder if the slow part is parsing these logical op expressions or if there is some other issue.</div>

</div>
</blockquote></div><br></div></div><div>If you do </div><div><br></div><div>$ clang -cc1 test/Sema/many-logical-ops.c</div><div><br></div><div>all the time is spent in SequenceChecker. If I uncomment "CheckUnsequencedOperations(E)" from Sema::CheckCompletedExpr() there is no slow down (also why does this run all the time without checking if the warning is enabled or not ?).</div>
<div><br></div><div>CC'ing Richard.</div></div></blockquote></div><br></div><div class="gmail_extra">Thanks for the analysis!</div></div>