Re: [coreboot] r3688 - in trunk/coreboot-v2/src: arch/i386/init cpu/amd/model_lx

[Jens Rottmann]

> > movb	%cl, %al
> > shrl	$2, %ecx
> > andb	$3, %al
> > rep	movsl
> > movb	%al, %cl
> > rep	movsb
>
> I guess we don't care that this movs 256 bytes extra when al & 3 = 0.

No, it doesn't. "rep movs" doesn't do anything if (e)cx = 0.

Regards,
Jens

r3688 - in trunk/coreboot-v2/src: arch/i386/init cpu/amd/model_lx

[Peter Stuge]

Jens Rottmann wrote:
> > I guess we don't care that this movs 256 bytes extra when al & 3 = 0.
>
> No, it doesn't. "rep movs" doesn't do anything if (e)cx = 0.

Nice!

//Peter