Fixed code snippet in Kaleidoscope tutorial to reflect final full code listing

[n0madsky]

Summary

In section 3.4 of the Kaleidoscope Tutorial, there are some code snippets relating to PrototypeAST::codegen()

Function *F =
  Function::Create(FT, Function::ExternalLinkage, Name, TheModule); <- note here

In the final Full code listing, this was changed to

Function *F =
    Function::Create(FT, Function::ExternalLinkage, Name, TheModule.get()); <-

This simple patch changes the first code snippet to be consistent with the second part.

[Wilfred]

Looks fine by me!

Do you need commit access? If you email Chris Lattner he'll give you access, or I can apply the patch.

[n0madsky]

Yes, this is my first LLVM patch and I would need commit access.

Although, for the sake of pragmatism and getting the patch in, I'd be very happy for you (or anyone else with commit access) to apply the patch.

docs/tutorial/LangImpl03.rst

.. code-block:: c++
     Function *PrototypeAST::codegen() {
       // Make the function type:  double(double,double) etc.
       std::vector<Type*> Doubles(Args.size(),
                                  Type::getDoubleTy(TheContext));
       FunctionType *FT =
         FunctionType::get(Type::getDoubleTy(TheContext), Doubles, false);
 
       Function *F =
-        Function::Create(FT, Function::ExternalLinkage, Name, TheModule);
+        Function::Create(FT, Function::ExternalLinkage, Name, TheModule.get());
 
 This code packs a lot of power into a few lines. Note first that this
 function returns a "Function\*" instead of a "Value\*". Because a
 "prototype" really talks about the external interface for a function
 (not the value computed by an expression), it makes sense for it to
 return the LLVM Function it corresponds to when codegen'd.
 
 The call to ``FunctionType::get`` creates the ``FunctionType`` that