Talk fixups - #107
Merged
Merged
Conversation
kulp
force-pushed
the
talk-fixups
branch
2 times, most recently
from
February 1, 2026 00:05
b6f901a to
6e6895f
Compare
If `rand` is found in the link order after the buffer, then the code may be too distant to reach without a type-3 (20-bit) offset. This too-distant relocation was meant to be caught by `tld`, but was not.
This belongs with fea8100. Until the current commit, and probably since that previous change, there was a buffer overflow.
Owner
Author
|
Beyond the specific changes for the talk, cf85f4a was also included, to correct an issue that shows up now, presumably because of recent Icarus Verilog updates. |
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
1 participant
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.
Collect fixes discovered while writing up a talk.