Skip to content

llgo: xgo-dev - #100

Merged
visualfc merged 1 commit into
goplus:mainfrom
visualfc:xgo-dev
Aug 17, 2026
Merged

llgo: xgo-dev#100
visualfc merged 1 commit into
goplus:mainfrom
visualfc:xgo-dev

Conversation

@visualfc

Copy link
Copy Markdown
Member

No description provided.

@visualfc
visualfc merged commit 1e12995 into goplus:main Aug 17, 2026
18 checks passed

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: replace goplus/llgo with xgo-dev/llgo

No blocking issues found. The change is a clean, complete find-and-replace of the LLGo module path, and I confirmed it is consistent across the repository.

Verified

  • All three occurrences updated: README.md:13 (compiler link) and the two //go:linkname directives at rtype_llgo.go:139 and rtype_llgo.go:379.
  • A repo-wide search for goplus/llgo returns zero remaining references, so nothing was missed.
  • linkname_go121.go:13 also declares an interequal linkname, but it targets the standard runtime.interequal under the !llgo build tag — correctly left unchanged.
  • go.mod is rightly untouched: the xgo-dev/llgo path appears only in //go:linkname directives (resolved at LLGo link time under the //go:build llgo tag) and a doc link, so it is not a tracked module dependency.

Non-blocking note (informational)

  • The two //go:linkname directives now pull unexported runtime symbols (interequal, DirectIfaceData) from a different GitHub org (xgo-dev vs goplus). Because these are cross-module pull-linknames into an internal/ runtime package, correctness depends on the target keeping identical signatures/ABI, and the linkage is not pinned by go.sum. If xgo-dev/llgo is the sanctioned successor home of LLGo in the XGo ecosystem, this is a benign rename — worth a quick confirmation of the new repo's ownership/governance before merging.

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.

1 participant