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

Opaque Pointers #421

Open
willtunnels opened this issue May 21, 2023 · 0 comments
Open

Opaque Pointers #421

willtunnels opened this issue May 21, 2023 · 0 comments

Comments

@willtunnels
Copy link
Contributor

willtunnels commented May 21, 2023

As of LLVM 15, LLVM is moving away from typed pointers and toward the use of opaque pointers. These are already supported in LLVM, though, for the time being, typed pointers are also still supported. You can read about this decision and the reasoning behind it here: https://llvm.org/docs/OpaquePointers.html.

We should evolved the Inkwell API to reflect this. At a minimum (and maybe this is all that needs to be done) we need:

  • A ptr_type function on Context
  • An is_opaque function on PointerType

We also might consider a deprecation warning in LLVM 15+ on typed pointer methods.

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

No branches or pull requests

2 participants