Re: [ros-dev] [ros-diffs] [tkreuzer] 42353: asm version of DIB_32BPP_ColorFill: - Add frame pointer - Get rid of algin_draw, 32bpp surfaces must be DWORD aligned - Optimize the loop - Add comments

[Timo Kreuzer]

Jose Catena schrieb:
>   A correction of my previous msg:
>  In asm I would write the loop as:
>         mov eax, iColor
>         mov ebx, pulLine
>         mov edx, cy
>  L1:
>         mov edi, ebx
>         mov ecx, _cx
>         rep stosd
>         add ebx, lDelta
>         dec edx
>         jnz l1
>  It is not possible to optimize the loop further AFAIK, and this only saves a
>  cmp and jnz in the outer loop, a tiny gain.
>     
That looks almost like the version I wrote, except I didn't use memory
access inside the loop, but registers, which should saves some cycles.