<div dir="ltr">Adding the list back in...<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 14, 2016 at 6:05 PM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Thu, Apr 14, 2016 at 5:52 PM, Rob Clark <span dir="ltr"><<a href="mailto:robdclark@gmail.com" target="_blank">robdclark@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Thu, Apr 14, 2016 at 7:23 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> wrote:<br>
><br>
> On Apr 14, 2016 4:15 PM, "Connor Abbott" <<a href="mailto:cwabbott0@gmail.com" target="_blank">cwabbott0@gmail.com</a>> wrote:<br>
>><br>
>> On Wed, Apr 13, 2016 at 7:45 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>><br>
>> wrote:<br>
</span><span>>> > I just rebased your patches (I grabbed the v2 from your <a href="http://freedesktop.org" rel="noreferrer" target="_blank">freedesktop.org</a>)<br>
>> > on<br>
>> > latest master and sedjob'd them to switch the order of parameters.  You<br>
>> > can<br>
>> > find it here:<br>
>> ><br>
>> ><br>
>> > <a href="https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/nir-foreach-block-v3" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/nir-foreach-block-v3</a><br>
>><br>
>> Ok, well if you're willing to do the work then I'm fine with it :)<br>
><br>
> It wasn't much work.  I just did git format-patch, ran sed on the patches,<br>
> and did git am.<br>
><br>
>> should I send out the modified patch 1, or will you, or...?<br>
><br>
> I don't care too much about a resend.  Just don't resend everything.  I<br>
> haven't really looked at them hard enough to give a review yet.  Too busy<br>
> trying to land a Vulkan driver. :-)<br>
><br>
> I'll see if I can't get them reviewed yet this week though.<br>
><br>
<br>
</span>jfyi, not sure what the odds of getting nir/gallium stuff landed in<br>
next week (still got a couple patches I guess I need to bribe someone<br>
to review)..<br>
<br>
but if jason's branch is close enough to final (at least from PoV of<br>
nir lowering passes) I could rebase on that and add squashable patches<br>
on to (which could either be squashed into original patches or added<br>
on top depending on what order things go)<br></blockquote><div><br></div></div></div><div>I've had it kicking around my brain long enough that I'm willing to say I like the concept and we should go forward with it. <br></div><div><br></div><div>Here's what i'd like to do:<br><br></div><div> 1) Merge the rename patch ASAP and we'll all rebase on that.  Feel free to add my R-B or I'll just add it and push.<br></div><div> 2) Merge all but the last patch as soon as we get the kinks worked out.<br></div><div> 3) Merge the last patch (that deletes nir_foreach_block_call) only after we've all had a chance to fix up our outstanding branches.<br><br></div><div>Seem reasonable to everyone?<span class="HOEnZb"><font color="#888888"><br></font></span></div><span class="HOEnZb"><font color="#888888"><div>--Jason <br></div></font></span></div></div></div>
</blockquote></div><br></div>