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.