Skip to content

Add LLVM 23 support - #28

Merged
vaivaswatha merged 2 commits into
vaivaswatha:masterfrom
adriaanjacobs:master
Sep 22, 2026
Merged

vaivaswatha merged 2 commits into
vaivaswatha:masterfrom
adriaanjacobs:master

Conversation

@adriaanjacobs

Copy link
Copy Markdown
Contributor

Hi

I'm still using DebugIR, still very useful!

I had to patch some things to get it to work on LLVM 23 (latest Ubuntu 24.04 default LLVM version).

This patch does not update the README or CI yet, I'll leave that up to you

@vaivaswatha

Copy link
Copy Markdown
Owner

Thank you @adriaanjacobs . I added the CI and README changes to your branch. I also removed pre LLVM-23 support.

@vaivaswatha
vaivaswatha merged commit 836c761 into vaivaswatha:master Sep 22, 2026
1 check passed
@adriaanjacobs

Copy link
Copy Markdown
Contributor Author

Ah, I had hoped you wouldn't. Thanks for merging, but would it be possible to try and keep support for other LLVM versions around?

I typically have projects targeting multiple different LLVM versions, and I expect many prototypes will continue targeting LLVM 15 for a while still (due to opaque pointers).

@vaivaswatha

Copy link
Copy Markdown
Owner

I think at some point we do need to drop support for older versions.

Anyway, for now, I have this: #29

@vaivaswatha

Copy link
Copy Markdown
Owner

I think at some point we do need to drop support for older versions.

Anyway, for now, I have this: #29

Do you want this tool itself building against the older LLVMs? because the tool (as far as I can tell) would just work for IR generated by older LLVMs anyway, even if it itself only builds with LLVM-23.

@adriaanjacobs

Copy link
Copy Markdown
Contributor Author

Ah, to clarify: all my uses of DebugIR is as an in-pass transformation that adds IR debug information to the compiled program (= debugir as a library, I am also the author of #21 for this reason), rather than as a standalone tool.

So in my case, the version of LLVM that DebugIR builds against matches the version of LLVM IR it is processing.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants