<div dir="ltr">I've been out of town on vacation. This patch is on my list to review as soon as I can.<div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 2, 2018 at 3:35 PM Yonghong Song <<a href="mailto:yhs@fb.com">yhs@fb.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Hi, David and Eric,<br>
<br>
Any opinion on this patch? Adrian mostly okay with the patch except<br>
some possible refactoring. It would be good to get your opinion so<br>
we can move the project forward.<br>
<br>
Thanks!<br>
<br>
Yonghong<br>
<br>
<br>
On 10/31/18 4:29 PM, Alexei Starovoitov via Phabricator wrote:<br>
> ast accepted this revision.<br>
> ast added a comment.<br>
> This revision is now accepted and ready to land.<br>
> <br>
> Eric, David, any objections to land it now?<br>
> It's orthogonal to everything else in the llvm land and we'll keep working on it for foreseeable future.<br>
> All comments were addressed and any further refactoring requests can be done later.<br>
> I'd like to avoid sitting on it for too long, since a bunch of folks in bpf community are waiting on it.<br>
> We have two large sets of kernel patches that depend on it.<br>
> We'd like to land llvm bits first, so our kernel test bots can do testing with latest llvm trunk and latest kernel tree asap.<br>
> <br>
> <br>
> Repository:<br>
>    rL LLVM<br>
> <br>
> <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736&d=DwIFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=Ew1MQ_MuKfrmZD6WfVqlNmbhAJNSt4WvT-AwUHD7Z0A&s=VDCS1P8iGI26lJfKjl6UVjUNIv0ap7jP8YrI1aivzZ4&e=" rel="noreferrer" target="_blank">https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736&d=DwIFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=Ew1MQ_MuKfrmZD6WfVqlNmbhAJNSt4WvT-AwUHD7Z0A&s=VDCS1P8iGI26lJfKjl6UVjUNIv0ap7jP8YrI1aivzZ4&e=</a><br>
> <br>
> <br>
> <br>
</blockquote></div>