Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to LLVM 5.1 #46

Closed
wants to merge 5 commits into from
Closed

Conversation

OndrejSlamecka
Copy link

@OndrejSlamecka OndrejSlamecka commented Dec 23, 2017

Hello, I am trying to follow the tutorial but needed to do some changes to make it work with LLVM 5.1. Besides minor things:

This is WIP. I welcome suggestions for improvement. So far I have changed only the code for chapter 3. I will change the rest of chapters' code and the tutorial text once we settle on the implementation.

@sdiehl
Copy link
Owner

sdiehl commented Dec 24, 2017

So I'm attempting to keep the buildable code upstream in the llvm-hs organization. And then just keep the prose in this repo. If you want to submit the patch up there, our CI is more up to date.

@OndrejSlamecka
Copy link
Author

OndrejSlamecka commented Dec 25, 2017

Sure thing, I am leaving this PR open to make changes to the text later. For the code, I opened the following PR

llvm-hs/llvm-hs-kaleidoscope#4

@OndrejSlamecka
Copy link
Author

OndrejSlamecka commented Dec 26, 2017

Closing in favour of likely future changes to the tutorial. See
llvm-hs/llvm-hs-kaleidoscope#4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LLVM 5.0 Error Calling Function
2 participants