Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unaligned access trickery #613

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Conversation

dkfellows
Copy link
Member

Live Packet Gatherer: A neater way to write the write_word() code. This results in two half-word store instructions being generated (which is the ideal case given that a full word store results in a bus error) while still being recognisably an actual write of a word.

@dkfellows dkfellows added enhancement This adds a new feature minor Not very important in progress Work in progress; known not ready to merge labels Jun 11, 2020
@dkfellows dkfellows self-assigned this Jun 11, 2020
@dkfellows dkfellows mentioned this pull request Jul 16, 2020
dkfellows added a commit to SpiNNakerManchester/sPyNNaker8 that referenced this pull request Aug 28, 2020
@dkfellows dkfellows marked this pull request as ready for review August 31, 2020 08:58
@alan-stokes
Copy link
Contributor

can this have a test suite associated with it. when anything RIPTMCS changes, i never trust it without full tests having gone over it

@dkfellows dkfellows added this to the 6.0.0 milestone Mar 30, 2021
@dkfellows dkfellows modified the milestones: 6.0.0, 7.0.0 Apr 12, 2021
@dkfellows dkfellows removed their assignment Sep 27, 2023
@dkfellows dkfellows added the C Relating to the C code in FEC label Sep 27, 2023
@dkfellows dkfellows modified the milestones: 7.0.0, 7.1.0 Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Relating to the C code in FEC enhancement This adds a new feature in progress Work in progress; known not ready to merge minor Not very important
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants