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><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">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>