Closed dpaoliello closed 1 year ago
I have a general policy against supporting multiple versions by adding LLVM version check ifdef code anyways (https://github.com/JuliaHubOSS/llvm-cbe/issues/60#issue-548626981), so looks good to me, and thanks for the PR!
Switching to opaque pointers would be the next big step. My guess is that it would be pretty minimal changes required to change all pointee types to void, except when they are required by operation itself, in which cases LLVM still has them (https://github.com/JuliaHubOSS/llvm-cbe/issues/154)
Switching to opaque pointers would be the next big step. My guess is that it would be pretty minimal changes required to change all pointee types to void, except when they are required by operation itself, in which cases LLVM still has them (#154)
Yep, I'm working on that right now.
Testing:
Note: I have not tested this against LLVM 10 - it should still compile, but the tests now make the assumption that we are using LLVM 16 (namely I'm setting options to disable opaque pointers).