Repository navigation
Conversation
The site's linker refused more than 32 inputs, a bound GNU ld does not have. The whole catalogue links 35 units on main, and the fullest coverage selections of a new ColdFire module refused on the count where native refuses for free space (64 mismatched refusals for Track Solo). The bound is now 256, a cap on one link's work, and a test links 40 units. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
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
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.
@repeat98, as you asked on the agent board.
What
linkColdFireRuntime(src/engine/coldfire-link.ts) refused more than 32 inputs. GNU ld has no such limit. The bound is nowMAX_LINK_INPUTS = 256, a cap on one link's work rather than a native limit.Why
On current
main(f577d629), with my own OS 1.40C:… not free, and the site refused earlier withThe ColdFire runtime has an invalid base or input count.Checks
coldfire-link.test.tslinks 40 units, checks that each is placed apart, and checks thatMAX_LINK_INPUTS + 1is still refused. It fails with the old cap and passes with the fix.npm run module:verify -- --all --os … --checkgives identical output with the old cap and with the fix.npm run pr:ready: Ready.Not part of this PR: that
--all --checkrun reports mismatches in most committed records onmainwith or without this change (mostly "the module-owned image differs from native outside the platform writes" and "the site would build different menus than native was given"). They look like records older than current main.Guides: nothing lacking for this change.
🤖 Generated with Claude Code