1) Two comments ";ASSERT(lp_quantization <= 31)" in the new functions 
..._wide_asm_ia32()
-- just to mention this constraint.
(max. possible value of lp_quantization is 15, so it's not a problem)


2) "mov cl, ..." was replaced with "mov ecx, ..." (again Agner Fog, 
optimizing_assembly.pdf)

summary: write to a partial register may result in false dependencies
between instructions, so it is better to avoid it.
(also bitreader_asm.nasm and stream_encoder_asm.nasm both have "mov ecx, ..."
instructions, and no "mov cl, ...").

Attachment: cl_to_ecx.patch
Description: Binary data

_______________________________________________
flac-dev mailing list
flac-dev@xiph.org
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to