Dan Kolsoi

I think your idea for `Phi` is very close to my plans for sub types in the second iteration of Inkwell (#8). I think it would make sense to have...

Yes, that is what I mean. I'd much rather focus on core functionality such as implementing all (or almost all) llvm methods, multi version compatibility, and documentation before working on...

`set_name` now lives on the `BasicValue` trait. I tried to moved `get_name` as well, but figuring out the right lifetime for the return (`&'??? CStr`) turned out to be a...

This might not actually be possible with modern rust. Apparently it might require VLA (variable length arrays), which do not exist today because they're easy to blow through the stack...

Related RFC issue: https://github.com/rust-lang/rfcs/issues/618

If we don't have rust support for this (and this does indeed turn out to be a significant speed issue - which it may not) we could add some sort...

https://github.com/rust-lang/rust/issues/48055 might be a better approach to this

Or https://github.com/HeroicKatora/static-alloc

Added experimental feature flag for testing this: https://github.com/TheDan64/inkwell/commit/673a3d58f39f957fb6ef509bf264d0ffdedc6665

I don't think add_global_mapping is the issue here specifically but rather rustdoc running doctests which contain certain subsets of LLVM. Like, I think builders and modules had issues but not...