Skip to content

Support older versions of LLVM - #29

Open
vaivaswatha wants to merge 4 commits into
masterfrom
older_llvm
Open

vaivaswatha wants to merge 4 commits into
masterfrom
older_llvm

Conversation

@vaivaswatha

Copy link
Copy Markdown
Owner

No description provided.

@vaivaswatha vaivaswatha self-assigned this Sep 22, 2026
@vaivaswatha vaivaswatha mentioned this pull request Sep 22, 2026
@vaivaswatha

Copy link
Copy Markdown
Owner Author

@adriaanjacobs can you confirm that this branch works for you? I'll merge it after that.

@adriaanjacobs

Copy link
Copy Markdown
Contributor

Just tested; it doesn't work on LLVM 15 because of the explicit version checks for at least LLVM 16. Would you like to drop support for 15 entirely, or would you be open to keeping it? I would be happy to contribute a patch that keeps support for 15 through 23.

I'm emphasizing LLVM 15 because it was the last version with non-opaque pointer support (albeit deprecated already).

@vaivaswatha

Copy link
Copy Markdown
Owner Author

Just tested; it doesn't work on LLVM 15 because of the explicit version checks for at least LLVM 16. Would you like to drop support for 15 entirely, or would you be open to keeping it? I would be happy to contribute a patch that keeps support for 15 through 23.

I'm emphasizing LLVM 15 because it was the last version with non-opaque pointer support (albeit deprecated already).

Can you check now? A full review of the PR would be helpful too. The CI changes that I added are mostly AI generated (especially the python test scripts). They seem to be working, but if you understand it well enough and have comments, do let me know.

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