<div dir="ltr"><br><div class="gmail_extra">On Tue, Apr 23, 2013 at 11:36 PM, Christopher Jones <span dir="ltr"><<a href="mailto:chrisjones.lambda@gmail.com" target="_blank">chrisjones.lambda@gmail.com</a>></span> wrote:<br>
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks a lot, Sean.  I'll keep that in mind like you suggested earlier. <div>
<br></div><div>Is my patch good?  Or do I need to do more? : )</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br></div></div></div></blockquote><div><br></div><div style>It looks fine. Keeping changes small and focused is good. That's a big part of LLVM development style: small, focused changes, with good justification that makes them easy to review. (the other major pillar of LLVM development style is excellent testing, but changes like these with no functionality change don't need tests).</div>
<div style><br></div><div style>I'd also like to emphasize that the ArrayRef-ization is really a *much* more important project. Every time you see pointer+length being passed as arguments, you can basically indiscriminately replace it with ArrayRef (or MutableArrayRef if it needs to be mutable). It's a huge win in virtually every case.</div>
<div style><br></div><div style>-- Sean Silva</div></div></div></div>