Page MenuHomePhabricator

[examples] Move llvm/examples/OCaml-Kaleidoscope/ to llvm-archive

Authored by xgupta on Feb 8 2021, 3:12 PM.



It seems Ocaml-Kaleidoscope/ tutorial is not[[ | updated from the last 10 years ]] (the last fix was on Fab 10, 2011). It does not validate itself with the tests. It seems no-one maintaining Ocaml. I tried to build it, Chapter 4,5,6,7 are not compiling because of an error In ExecutionEngine.

The error was:-

File "", line 37, characters 25-53:
37 | let result = ExecutionEngine.run_function the_function [||]


Error: Unbound module ExecutionEngine
Command exited with code 2.

Diff Detail

Unit TestsFailed

70 msx64 debian > Polly.ScopInfo::user_provided_assumptions.ll
Script: -- : 'RUN: at line 1'; opt -polly-process-unprofitable -polly-remarks-minimal -polly-use-llvm-names -polly-import-jscop-dir=/mnt/disks/ssd0/agent/llvm-project/polly/test/ScopInfo -polly-codegen-verify -pass-remarks-analysis="polly-scops" -polly-scops -disable-output < /mnt/disks/ssd0/agent/llvm-project/polly/test/ScopInfo/user_provided_assumptions.ll 2>&1 | FileCheck /mnt/disks/ssd0/agent/llvm-project/polly/test/ScopInfo/user_provided_assumptions.ll

Event Timeline

xgupta created this revision.Feb 8 2021, 3:12 PM
xgupta requested review of this revision.Feb 8 2021, 3:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 8 2021, 3:12 PM
whitequark accepted this revision.Feb 8 2021, 3:49 PM

LGTM. I've been maintaining OCaml bindings for a while, but I never got the time to update the tutorial. (Ironically, that tutorial was what got me into LLVM *and* OCaml...) It is regrettable that it will be removed but I don't see another solution.

This revision is now accepted and ready to land.Feb 8 2021, 3:49 PM

OK, Thanks @whitequark for taking the time to review and comment. It is really unfortunate to remove such an awesome tutorial.

xgupta retitled this revision from [Draft] [examples] Move llvm/examples/OCaml-Kaleidoscope/ to llvm-archive to [examples] Move llvm/examples/OCaml-Kaleidoscope/ to llvm-archive.Feb 8 2021, 10:19 PM
geexie added a subscriber: geexie.Jan 12 2022, 5:41 AM