<div dir="ltr">Clang-formated in r245472.<div><br></div><div>- Lang.</div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 18, 2015 at 11:13 AM, Lang Hames <span dir="ltr"><<a href="mailto:lhames@gmail.com" target="_blank">lhames@gmail.com</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">> <span style="font-size:13px">It does look like this adds a long line or two though - do you mind </span><span style="font-size:13px">clang-format'ing the patch?</span><div><span style="font-size:13px"><br></span></div><div><span style="font-size:13px">Clang-formatting the tutorials is a good idea. I've applied this patch in r245322 - I'll follow up with a clang-format patch next.</span></div><div><span style="font-size:13px"><br></span></div><div><span style="font-size:13px">- Lang.</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 17, 2015 at 10:03 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Lang Hames via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> writes:<br>
> Hi All,<br>
><br>
> Attached is a patch to partially C++11'ify the Kaleidoscope tutorials. There<br>
> are a few LLVM coding convention fixes and range-based for loops in here, but<br>
> the bulk of the patch is concerned with switching raw pointers to unique_ptr.<br>
><br>
> This is a significant change from the original Kaleidoscope tutorials which<br>
> deliberately avoided any memory management so as not to overcomplicate the<br>
> examples. I thought it was time to revisit this idea, since C++11 has removed<br>
> a lot of the complication of C++ memory management, and there's a benefit to<br>
> having the tutorial code more closely resemble the rest of the LLVM codebase. <br>
><br>
> Any thoughts/feedback on this direction?<br>
<br>
This is great - it isn't really harder to understand and it's that much<br>
closer to what the code you'll actually want to write after reading the<br>
tutorial. LGTM.<br>
<br>
It does look like this adds a long line or two though - do you mind<br>
clang-format'ing the patch? Actually, a lot of this code predates our<br>
formatting rules and you're touching half of the lines anyway, so it's<br>
probably reasonable to just clang-format the whole thing in a prep patch<br>
before this one if you feel like it.<br>
<br>
> I see this as a first step towards fully modernizing the Kaleidoscope<br>
> tutorials. My next step will be to update Chapter 4 onwards to bring them up<br>
> to speed with the recent JIT API changes - I've already started work on a<br>
> follow-up patch for this. If anybody else is interested in helping me bring<br>
> these tutorials up to speed let me know - assistance will very welcome.<br>
><br>
> - Lang.<br>
><br></blockquote></div></div></blockquote></div></div></div></div>