PleasingFungus / Silicon-Zeroes

Issue repository for Silicon Zeroes. (Contains no actual code.)
12 stars 0 forks source link

Please consider strengthening tests for branch insns #118

Open amonakov opened 7 years ago

amonakov commented 7 years ago

Some puzzles on the last board where the player needs to implement a (conditional) branch instruction (e.g. "Delay Branch" and "Finale") seem to be missing tests with consecutive branch instructions. It should be a special case for most pipeline schemes implemented by players, so including such test scenarios might be useful to help players encounter their first "serious" bug in the mock CPU! :)

(does the current module count par account for handling such branches?)

PleasingFungus commented 7 years ago

Not sure if it'll make it into this release, but I'll take a look!

PleasingFungus commented 7 years ago

Added an untaken branch in the Delay Branch custom test in db9d27a803. Finale to come.

Cyprex commented 6 years ago

(does the current module count par account for handling such branches?)

You can solve "Delay Branch" in 21 Modules/ 252 Ticks(Par: 22/242), i can post a solution if you want.

ghost commented 5 years ago

(does the current module count par account for handling such branches?)

You can solve "Delay Branch" in 21 Modules/ 252 Ticks(Par: 22/242), i can post a solution if you want.

if you wouldn't mind I need "delay branch", "reg hazard" and "Finale".