<div dir="ltr">OK, then let me try to see why we are getting a wrong result for stream 0 first.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 3, 2016 at 6:04 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yea it's ok, but it seems like the same amount of work to do the patch either way, but the other way adds more functionality.  If you plan to do split fpm immediately after this patch I guess I can't complain, but if you're planning to work on something else next then we should just do this now so it doesn't get neglected <br><div class="HOEnZb"><div class="h5"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 3, 2016 at 5:52 PM Rui Ueyama <<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ruiu added a comment.<br>
<br>
I guess incremental patches are a preferred way, so in either way, don't you think submitting this is a good idea?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D23145" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23145</a><br>
<br>
<br>
<br>
</blockquote></div>
</div></div></blockquote></div><br></div>