<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="font-family:arial,sans-serif;font-size:13px">I will follow the LLVM developers policy, which is to submit/commit small<br>
</span><span style="font-family:arial,sans-serif;font-size:13px">incremental patches.  Unfortunately, I don't believe it's acceptable to<br></span><span style="font-family:arial,sans-serif;font-size:13px">bend the rules to accommodate our needs.</span></blockquote>
<div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><font face="arial, sans-serif">Hi Chad,</font></div><div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">I think Jiangning was more wondering whether it was possible for you to send him (or post to the list, but not ask for review on) a monolithic patch with all your changes applied - not asking you to change how the reviews are being done!</font></div>
<div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">Cheers,</font></div><div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif">James</font></div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On 9 October 2013 16:45, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Chad,<br>
<div class="im"><br>
> Alternatively, I could add the implicit zero to Ops[] in CGBuiltin.cpp,<br>
> but for some reason that doesn't feel right either.  Maybe it's fine.<br>
<br>
</div>I'd say that's the nicer solution. For example it means that if anyone<br>
wrote a vceq_s16 (say) and much later on we could prove the argument<br>
was a constant 0 then we'd still be able to use the "cmeq hN, #0"<br>
form.<br>
<br>
Of course, we could add patterns to do that anyway, but I like the<br>
goal of having a minimal set of extra intrinsics. If nothing else<br>
there's a reasonable hope that it would give more of this<br>
serendipitous cross-matching.<br>
<div class="im"><br>
>> Possibly modelled as a NEON_VDUP for selection?<br>
><br>
> I'm not sure I follow.  I'm rather new to this area.<br>
<br>
</div>Just musing on what would get through to ISelDAG if you called<br>
"@llvm.aarch64.neon.vcmeq.v1i16(<1 x i16> %lhs, <1 x i16> <i16 0>)".<br>
I'm not really sure myself, though.<br>
<div class="HOEnZb"><div class="h5"><br>
Cheers.<br>
<br>
Tim.<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>