[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.