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

Comparison to inkwell #1

Open
dlight opened this issue Jun 4, 2023 · 3 comments
Open

Comparison to inkwell #1

dlight opened this issue Jun 4, 2023 · 3 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@dlight
Copy link

dlight commented Jun 4, 2023

How does Dorian compares to inkwell? Maybe a short section on readme would be nice.

@peterhenryd
Copy link
Owner

peterhenryd commented Jul 20, 2023

Simply put, Dorian is to not trying to wrap LLVM, but rather be a very intuitive abstraction of it; will make sure to add something like you suggested soon. Thanks!

Moreover, if one were to create a vendiagram between what Dorian and Inkwell are trying to achieve (as far as I'm aware), the differences as of now aren't especially significant, but that's likely to evolve in the future as this project progresses.

@peterhenryd peterhenryd added the documentation Improvements or additions to documentation label Jul 20, 2023
@peterhenryd peterhenryd self-assigned this Jul 20, 2023
@dlight
Copy link
Author

dlight commented Jul 20, 2023

Dorian is to not trying to wrap LLVM, but rather be a very intuitive abstraction of it

You mean that Inkwell's API may be more feature complete than Dorian (since it aims to prove the entire llvm API surface), but Dorian would be easier to use?

I mean, Inkwell is also an abstraction I think (but it may be lower level).

Since Dorian is higher level, does it make sense to add Inkwell as a dependency and write Dorian on top of Inkwell? I mean currently Dorian uses llvm-sys but as such it has a lot of unsafe code.

@peterhenryd
Copy link
Owner

You are correct. It may be a good idea to build Dorian on top of Inkwell, especially since it would be much easier to leverage different LLVM versions, and would likely speed up development a lot. I'll create a new branch and mess around with it.

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

No branches or pull requests

2 participants