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

Adapteva zigzag tests now working #37

Merged
merged 2 commits into from
Jul 31, 2016
Merged

Adapteva zigzag tests now working #37

merged 2 commits into from
Jul 31, 2016

Conversation

snim2
Copy link
Member

@snim2 snim2 commented Jul 31, 2016

No description provided.

@snim2 snim2 self-assigned this Jul 31, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 99.903% when pulling 5c2e93f on zigzag into 0a5d8c8 on master.

…of the chip. Adapteva zigzag tests now work correctly.
This makes relatively little difference to the simulator performance.
Copying memory is the time consuming part of the elf_loader.
Rough timings (on laptop):

BEFORE:
ELF file loader took: 2.351138s
ELF file loader took: 2.191296s
ELF file loader took: 2.215894s
ELF file loader took: 2.204229s
ELF file loader took: 2.255121s

AFTER:
ELF file loader took: 2.255121s
ELF file loader took: 2.111252s
ELF file loader took: 2.120284s
ELF file loader took: 2.177955s
ELF file loader took: 2.148350s
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7850719 on zigzag into 0a5d8c8 on master.

@snim2 snim2 merged commit a4dea0d into master Jul 31, 2016
@snim2 snim2 deleted the zigzag branch July 31, 2016 17:32
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.

None yet

2 participants