<div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small">Rafael,</div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small">Your patch looks good. (LGTM). I don't yet have committer privileges, but I will try and find someone here to commit it. I will then rebase the CL and resend it.</div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small">Karl</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">Karl Schimpf <br></div></div>
<br><div class="gmail_quote">On Fri, Jun 19, 2015 at 7:52 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">invalid-data-after-module.bc is not used.<br>
<br>
The patch has not been git-clang-formated.<br>
<br>
It looks like there is an independent part of the patch with is just<br>
about removing IsStreamed. I have extracted it to the attached patch.<br>
<br>
It has a small but noticeable impact on performance. I timed running<br>
"opt -disable-output -disable-verify" on a ltoed clang. It goes from<br>
<br>
<a href="tel:14.752845231" value="+14752845231">14.752845231</a> seconds time elapsed<br>
    ( +-  0.16% )<br>
<br>
to<br>
<br>
15.012463721 seconds time elapsed<br>
    ( +-  0.11% )<br>
<br>
It is small enough that I think it is worth the dropping of code path<br>
differences between the two cases.<br>
<br>
Do you agree with the attached patch? If so, please commit it first<br>
and rebase the rest of your patch.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>