[LLVMbugs] [Bug 10016] New: Missing line in OCaml Kaleidoscope tutorial

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=10016

           Summary: Missing line in OCaml Kaleidoscope tutorial
           Product: Website
           Version: unspecified
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: Documentation
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: damien.schoof at gmail.com
                CC: llvmbugs at cs.uiuc.edu


I've just been working through the Kaleidoscope tutorial online at
http://llvm.org/docs/tutorial/OCamlLangImpl2.html#parserbinops and I've found a
missing line in the section about parsing binary operators (linked).
The line of code (along with the corresponding comment) is:

        (* Parse the primary expression after the binary operator. *)
        let rhs = parse_primary stream in

found in lines 63 and 64 of parser.ml.
This line is a very important part of the code, and it is impossible to
understand the flow of logic without them in the tutorial.  The tutorial
already mentions this line (below the relevant snippet) by saying "and then
parses the primary expression that follows."

All that is needed to fix this bug is to add the 2 lines of code in, and things
will make much more sense.
This bug applies only to the OCaml version of the tutorial, the C++ version is
fine.

-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.