<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 08/25/2015 05:39 PM, Andy Somogyi
via llvm-commits wrote:<br>
</div>
<blockquote
cite="mid:9E5E9F86-6C68-40AD-9E52-E9E77AB52552@gmail.com"
type="cite">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<div>I think this should go in the 3.7 release.</div>
</blockquote>
I disagree. I'd be fine with a forward reference (e.g. "This
document is out of date, we suggest you read the one for ToT."), but
this would be a large change to make at the last minute. <br>
<blockquote
cite="mid:9E5E9F86-6C68-40AD-9E52-E9E77AB52552@gmail.com"
type="cite">
<div><br>
</div>
<div>The kaleidoscope tutorials have been broken for a long time,
ever since the removal of the regular JIT, and nearly everyone I
know used the Kaleidoscope tutorials to learn LLVM, they are a
very good resource and they simply should work. <br>
<br>
Sent from my <span style="-webkit-tap-highlight-color: rgba(26,
26, 26, 0.292969); -webkit-composition-fill-color: rgba(175,
192, 227, 0.230469); -webkit-composition-frame-color: rgba(77,
128, 180, 0.230469); ">iPhone</span></div>
<div><br>
On Aug 25, 2015, at 6:35 PM, Eric Christopher via llvm-commits
<<a moz-do-not-send="true"
href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>>
wrote:<br>
<br>
</div>
<blockquote type="cite">
<div>
<div dir="ltr">This looks pretty great.
<div><br>
</div>
<div>Couple of comments:</div>
<div><br>
</div>
<div>Man, would the renaming have been nice to do
before/after :)</div>
<div><br>
</div>
<div>
<div>+simple: The KaleidoscopeJIT has a straigtforward
symbol resolution rule that</div>
</div>
<div><br>
</div>
<div>Typo.</div>
<div><br>
</div>
<div>Any reason not to have the KaleidoscopeJIT file in the
tutorial?</div>
<div><br>
</div>
<div>
<div>+ : TM(EngineBuilder().selectTarget()),
DL(TM->createDataLayout()),</div>
</div>
<div><br>
</div>
<div>As far as the data layout, this hurts. Can we get it
from the module at use time?</div>
<div><br>
</div>
<div>Otherwise LGTM and hopefully we can just pull this into
3.7 if we have another RC. Otherwise we probably want to
hold off?</div>
<div><br>
</div>
<div>-eric</div>
<div><br>
</div>
<div><br>
</div>
<div>
<div>
<div class="gmail_quote">
<div dir="ltr">On Tue, Aug 25, 2015 at 2:01 PM Lang
Hames <<a moz-do-not-send="true"
href="mailto:lhames@gmail.com">lhames@gmail.com</a>>
wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Hi All,
<div><br>
</div>
<div>As some of you have noticed, Kaleidoscope has
had some issues since we removed the legacy JIT
last year. The tutorial code had been relying on
the behavior of the legacy JIT, and the switch
to MCJIT caused many simple use-cases to break
(e.g. repeat calls to functions). Kaleidoscope
Chapter 4, which introduces JIT support, had
been largely fixed by taking some of the code
from Andy Kaylor's Kaleidoscope/MCJIT tutorials,
but this added a lot of engineering detail to
what was supposed to be a beginner's tutorial.
The later chapters were never updated.</div>
<div><br>
</div>
<div>The attached patch fixes this situation by
updating all of the Kaleidoscope tutorials to
use a new, custom, ORC-based JIT:
KaleidoscopeJIT. By using this instead of MCJIT,
all of the original Kaleidoscope functionality
is restored, and we actually do a better job of
behaving like a REPL (for example, functions can
be redefined). Assuming this patch is accepted,
in the near future I hope to add one or two new
chapters to Kaleidoscope that involve modifying
the KaleidoscopeJIT to support lazy compilation
(This will mostly involve merging code from the
Kaleidoscope/Orc tutorials).</div>
<div><br>
</div>
<div>Most of this patch is concerned with the
switch to KaleidoscopeJIT and corresponding
updates to the documentation. In particular,
Chapter 4 has been substantially updated. There
is also a little bit of C++11 modernization and
general cleanup.</div>
<div><br>
</div>
<div>All comments welcome. If anyone would like to
road-test the new tutorials I'd be particularly
interested to hear feedback on that. </div>
<div><br>
</div>
<div>Cheers,</div>
<div>Lang.</div>
<div><br>
</div>
</div>
</blockquote>
</div>
</div>
</div>
</div>
</div>
</blockquote>
<blockquote type="cite">
<div><span>_______________________________________________</span><br>
<span>llvm-commits mailing list</span><br>
<span><a moz-do-not-send="true"
href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a></span><br>
<span><a moz-do-not-send="true"
href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a></span><br>
</div>
</blockquote>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
llvm-commits mailing list
<a class="moz-txt-link-abbreviated" href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>
<a class="moz-txt-link-freetext" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a>
</pre>
</blockquote>
<br>
</body>
</html>