go-llvm: add LLVM 23 support and make it default - #2
Merged
Conversation
We still haven't updated TinyGo, so this change is needed so we can use tinygo-org#73 in TinyGo.
* Add LLVM 22 build support Written entirely by Claude (Anthropic's Claude Code), at the request of and under the direction of dgryski, as part of a broader effort to get TinyGo building against upcoming LLVM releases. Adds the llvm22 build-tag config file (mirroring the existing llvm21 one) and a CI matrix entry for it. Also adds a regression test confirming that LLVM 21+'s 'captures' parameter attribute (which replaced the boolean 'nocapture' enum attribute) round-trips correctly through the existing generic enum-attribute API, with value 0 corresponding to captures(none) -- no binding/C++ shim changes were needed for this, since 'captures' is an IntAttr using the same wire format as e.g. 'align' or 'dereferenceable'. Verified by running the full test suite against real LLVM 22.1.8 headers and libraries (via Homebrew). * limit captures_test to llvm 21 and above. * Fix switch-instruction case-value access for LLVM 22 Written entirely by Claude (Anthropic's Claude Code), at the request of and under the direction of dgryski, on the dgryski/llvm23 branch (LLVM 22 support, in preparation for the eventual LLVM 23 release). LLVM 22 stopped exposing SwitchInst case values as regular instruction operands -- only the condition and destination-block operands remain as operands now. Case values must be read via the new LLVMGetSwitchCaseValue C API added in the same release. This was discovered while chasing a real-world regression in TinyGo's interp package (see the paired tinygo commit on the same branch), which manually walked switch operands assuming the old layout; verified independently here first via a minimal standalone .ll repro against real LLVM 20/21/22 headers and libraries, isolating the exact version cutoff (21 -> 22) before touching any tinygo code. Adds: - Value.SuccessorsCount()/Value.Successor(i) in ir.go, using the stable, version-independent LLVMGetNumSuccessors/LLVMGetSuccessor API to enumerate a switch's destination blocks (successor 0 is always the default destination; successors 1..N-1 are the cases). - Value.GetSwitchCaseValue(i), version-gated: switch_llvm22.go uses the new LLVMGetSwitchCaseValue API, switch_pre22.go falls back to the old operand-based read for LLVM <22. - switch_test.go, a regression test verifying case values and destinations round-trip correctly; passes against real LLVM 20/21/22.
…VM 22 LLVM 22 deprecated 8 global-context C API functions in favor of their context-aware counterparts. Instead of suppressing the warnings, call the non-deprecated InContext APIs directly. The non-context Go functions (MDKindID, StructType, ConstString, ConstStruct, AddBasicBlock, InsertBasicBlock) now delegate to GlobalContext().<method>(), which already calls the InContext C APIs. IntPtrType calls LLVMIntPtrTypeInContext directly. Only LLVMGetGlobalContext still needs a warning-suppression wrapper in deprecated.c since it has no non-deprecated replacement. The context-aware C APIs have been available since well before LLVM 14, so this works across all supported LLVM versions. Signed-off-by: deadprogram <ron@hybridgroup.com>
zhouguangyuan0718
force-pushed
the
codex/llvm23-integration
branch
from
July 26, 2026 16:54
81f6292 to
1b722f8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
staticllvmtagValidation
go test -count=1 -tags="byollvm llvm23" ./...against the local LLVM 23.1 shared librarygo test -count=1 ./...against the same library