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

Improve evaluator / find new LLVM wrapper #14

Open
chemix-lunacy opened this issue Mar 5, 2024 · 0 comments
Open

Improve evaluator / find new LLVM wrapper #14

chemix-lunacy opened this issue Mar 5, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@chemix-lunacy
Copy link
Contributor

chemix-lunacy commented Mar 5, 2024

Inkwell support for certain things was patchy and some of the getelementptr features had to be done directly via LLVM-sys.

Things which need to be fixed:

  1. getelementptr needs to be centralized and made to work. Can't get a GEP instruction or follow it in Inkwell when it's an argument without dropping to the LLVM-sys object.
  2. Get the SSA identifier from instructions and values without having to to-string an instruction and regex (very hacky and never liked it).
  3. Refine as_value results building for composites and arrays.
  4. Fix eval_insertvalue index retrieval via regex. Seems to be a theme here.
@chemix-lunacy chemix-lunacy added the enhancement New feature or request label Mar 5, 2024
@chemix-lunacy chemix-lunacy changed the title Update Inkwell/llvm-sys or switch LLVM wrappers Improve evaluator / find new LLVM wrapper Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant