search

TheRainDoodle / Phenom-II-Benchmark

Small set of low level benchmarks for testing hardware speed against a Phenom II 810 Quad Core
GNU General Public License v3.0
91 stars 20 forks source link

Update SHR_REG_CL.asm #9

Open gmpassos opened 3 years ago

gmpassos commented 3 years ago

CL register wasn’t initialized before SHR call. Changed to value 1.

gmpassos commented 3 years ago

You will need a new benchmark video! ;-P

igavelyuk commented 3 years ago

Super! Will merge your repo to get it earlier :neckbeard: :see_no_evil:

gmpassos commented 3 years ago

You where performing a random number of shifts, depending of the dirty in the register value (not initialized). This will produce a different number of operations, a not valid benchmark, since you are comparing different scenarios.

gmpassos commented 3 years ago

You can create a video about benchmarks, and how is difficult to create a scenario that is always reproducible and fair for all tested sides. This issue is a good example of how the same code can produce different scenarios and how is difficult to ensure fairness.