search

JuliaHubOSS / llvm-cbe

resurrected LLVM "C Backend", with improvements
Other
846 stars 146 forks source link

Fixes to make the code compile again #55

Closed ctbur closed 4 years ago

ctbur commented 5 years ago

While following the instructions in the readme there were two compilation errors. This PR contains the changes needed to make the C backend compile.

sapir commented 5 years ago

Which version of LLVM are you using?

ctbur commented 5 years ago

Ok, I thought I was on branch release_80, but after checking it seems that I am actually on master. That's what I get for copying multiple lines of shell code without carefully checking the result..

In any case, the code compiles and the tests run successfully. Maybe you can use this in the future, otherwise you can close the PR.

sapir commented 5 years ago

Maybe make these changes with an #ifdef for LLVM version 9?

ctbur commented 5 years ago

LLVM 9 is released now, so the PR should be applicable.

vtjnash commented 4 years ago

Can you update the README to say this now targets LLVM 9, and we can merge this.

woachk commented 4 years ago

Please don't break prior LLVM versions in the process (talking about that second commit, perhaps using an #ifdef there?).