<table border="1" cellspacing="0" cellpadding="8">
    <tr>
        <th>Issue</th>
        <td>
            <a href=https://github.com/llvm/llvm-project/issues/62411>62411</a>
        </td>
    </tr>

    <tr>
        <th>Summary</th>
        <td>
            Inconsistency in Kaleidoscope tutorial
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            new issue
      </td>
    </tr>

    <tr>
      <th>Assignees</th>
      <td>
      </td>
    </tr>

    <tr>
      <th>Reporter</th>
      <td>
          perlun
      </td>
    </tr>
</table>

<pre>
    Hi,

In [the Kaleidoscope tutorial](https://llvm.org/docs/tutorial/MyFirstLanguageFrontend), there is a slight inconsistency e.g. on https://llvm.org/docs/tutorial/MyFirstLanguageFrontend/LangImpl04.html.

I.e. this (old form, seems to work with e.g. LVVM 14):

```c++
      // Get the symbol's address and cast it to the right type (takes no
      // arguments, returns a double) so we can call it as a native function.
 double (*FP)() = (double (*)())(intptr_t)ExprSymbol.getAddress();
 fprintf(stderr, "Evaluated to %f\n", FP());
```

vs this (new form):


```c++
      // Get the symbol's address and cast it to the right type (takes no
      // arguments, returns a double) so we can call it as a native function.
      double (*FP)() = ExprSymbol.getAddress().toPtr<double (*)()>();
      fprintf(stderr, "Evaluated to %f\n", FP());
```

I looked into this and this seems to be a consequence of the changes in 8b1771bd9f304be39d4dcbdcccedb6d3bcd18200; that patch edited the `toy.cpp` files, but not the associated files (https://github.com/llvm/llvm-project/edit/main/llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl04.rst and others).

This would be a "good first issue" since its a quite simple change, merely requiring to go through the relevant `LangImplxx.rst` files and compare with the changes in 8b1771bd9f304be39d4dcbdcccedb6d3bcd18200. Since the fix was non-trivial (=more than editing a single file) I am not planning to look into it myself, so feel free to pick this up if anyone feels like fixing it. :+1: 
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJzcVk-P27YT_TT0ZRBBouQ_Ovjg3Y1-v0UTIECKXAuKHEnsUqSWHNnrb1-Qsp3dJg3QoL1UEGxBHM28eW9mSBGC7i3inq3v2PphJWYanN9P6M1sV61T5_3_NeP3LH9g-WH5fbTA1nc0IPwiDGrlgnQTAs3kvBaGrR8Y3w1EU2DlgfGG8caY45g53zPeKCcD483Nmjcfz432gT4I28-ix8Y7S2gV4zXj90ADegQdQEAwuh8ItJXOBh0IrTwDZn0GzsI_EbCJrx7HyeRVNtBosjdpZ5gBDToA4ztnFHTOjxFhQBwDkIOT809w0jQsoD58-fIRiirmUR5ee2KbfLkl43fxTm8hXQt8-B9SzBzCeWydYXwbQCjlMQQQVoEUgUBTDBqtfOKFzhNGbCSeMIB133ErfD-PaClE3B5p9jYSq9zcGmS8huDghCCFBSmMiSFENLCC9BGhm60k7eyFl8t3MSbjh-ZTUmwX3bDyIb59s35bXR60pYn8b8R4_f5l8p9TolmPdFjyvNiWV3a6yWtLHeO7QAq9jxkwzt8fhZkFoYpcML7u2PreMs7jckR0DXj1c-P-tSDHcBPW4uki7Deq_be0S9cPBfyBLBm5T-RZef8XCpfv_yxfuv5NDR_BOPeECrRNzOqF7fRw69AWQUCcHvg8o5UIrksiyEHYHgNoC7u22G6LVtVdmVctlrWqlGyVlBJVu1FlK1Wx43nOyjugQRBMguQAqHRKYEBgm5zcOZPTxDY5dNpgkqydCaxbSkOE4KROKad1-GZi9pqGuc2kGy_T7PL3bvLud5TEeBNDMt6MQtuvNj857XygRJeL4zZEhV9z-2sk8eRmoxYGGee9cxF7_E6HMCPjHIKOlGqKVfc8a0IIepzMld5IwogezRk8Ps_aa9tHUfool3dzPywNgQaPwlLk8Yrw5SUivNG59JEbJ-FxGbg_KWIGnxPm-HmnX-AkYvPZd-T1UQuT6rp8GJ2PJsImlSNqEXPtDSY4sV8eQYxJ3ckIay-JxYJcylETjOeApkv7hYMO0UDnEaPZpOXTUqfzBLoDYc_OYrIJYPRTghZdasogFchdwcoDrNS-VHVZixXui82Ob3ixq-rVsJdVp6qykp1QG1lvtxvJW76p1-uyKkVRFCu95zkv84pvi916y-tsU8m2y7tS1KIuRduyKsdRaJNd99FVEnm_4VVRrIxo0YR0ZOA8DsxrBcQThN-nMm3nPrAqNzpQ-OqFNBncP77ZwLX9_jliNXuz_9tNkaDEDkhQ_wgAAP__SgLHxg">