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.

Sign up for GitHub

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

Jump to bottom

Update to official inkwell crate #151

Closed
baszalmstra opened this issue May 1, 2020 · 3 comments · Fixed by #254
Closed

Update to official inkwell crate #151

baszalmstra opened this issue May 1, 2020 · 3 comments · Fixed by #254
Labels
exp: high Achievable by investing significant time and effort, potentially through collaboration pri: high An issue resulting in complete or substantial loss of functionality, that can be circumvented type: build Changes that affect the build system or external dependencies
Milestone
Mun v0.3.0

Comments

@baszalmstra
Copy link
Collaborator

Currently, Mun uses LLVM 7.x.

It is beneficial for us to update to the latest LLVM but this requires some work to be compatible with the latest Inkwell.
@Wodann Wodann modified the milestone: Mun v0.3 May 1, 2020
@Wodann
Copy link
Collaborator

Wodann commented May 17, 2020

@joshuawarner32 has commenced work on this

@baszalmstra baszalmstra added this to the Mun v0.3.0 milestone May 21, 2020
@Wodann Wodann added pri: high An issue resulting in complete or substantial loss of functionality, that can be circumvented type: perf Changes that improve performance exp: high Achievable by investing significant time and effort, potentially through collaboration labels May 22, 2020
@joshuawarner32 joshuawarner32 mentioned this issue May 23, 2020
Closed
@baszalmstra
Copy link
Collaborator Author

I started working on this again!

@baszalmstra
Copy link
Collaborator Author

#254 updates the code to use the official inkwell repo, but we could not yet upgrade to the latest LLVM (10 as of writing) because of a bug in LLD: https://reviews.llvm.org/D86401 once this is solved we can upgrade to LLVM10+.

However, I'm renaming this issue because upgrading has a much lower priority than the inkwell update.

@baszalmstra baszalmstra changed the title Update to latest LLVM Update to official inkwell crate Aug 23, 2020
@baszalmstra baszalmstra mentioned this issue Aug 23, 2020
Merged
@Wodann Wodann added type: build Changes that affect the build system or external dependencies and removed type: perf Changes that improve performance labels Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
exp: high Achievable by investing significant time and effort, potentially through collaboration pri: high An issue resulting in complete or substantial loss of functionality, that can be circumvented type: build Changes that affect the build system or external dependencies
Projects
None yet
Milestone
Mun v0.3.0
Development

Successfully merging a pull request may close this issue.

refactor: upgrade to official inkwell baszalmstra/mun
2 participants
@baszalmstra @Wodann