Sounds good to me.<br>I'll review in detail later today<br><div class="gmail_quote">On Thu Nov 06 2014 at 2:03:14 PM Balaram Makam <<a href="mailto:bmakam@codeaurora.org">bmakam@codeaurora.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">All,<br>
<br>
I updated my comment with slowdowns in compilation times and other benchmarks that improve with this patch. Slowdowns in compile times are in noise range I do not see any performance regressions greater than 3% in Spec. Based on this data, I would like to get rid of the fast path and will upload a patch removing out the commented-out code if you agree. Thanks for reviewing.<br>
<br>
<a href="http://reviews.llvm.org/D6103" target="_blank">http://reviews.llvm.org/D6103</a><br>
<br>
<br>
</blockquote></div>