<div dir="ltr">Cool.  I'll focus on ArrayRef with a side of StringRef if I see them.  I guess now the patch gets reviewed and added (or since it's fairly trivial, maybe not?  Still learning here.)</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, Apr 24, 2013 at 10:11 AM, Sean Silva <span dir="ltr"><<a href="mailto:silvas@purdue.edu" target="_blank">silvas@purdue.edu</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"><div class="im">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><div class="gmail_quote"><div class="im"><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><div><div class="gmail_extra"><br></div></div></div></blockquote><div><br></div></div><div>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><br></div><div>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>
<span class="HOEnZb"><font color="#888888">
<div><br></div><div>-- Sean Silva</div></font></span></div></div></div>
</blockquote></div><br></div>