Diego Biurrun <[email protected]> writes: > On Mon, Aug 06, 2012 at 12:36:28AM +0100, Mans Rullgard wrote: >> nasm does not support 'CPU foonop' directives. This adds a configure >> test for the directive and uses it only if supported. >> --- >> configure | 2 ++ >> libavutil/x86/x86inc.asm | 12 +++++++++--- >> 2 files changed, 11 insertions(+), 3 deletions(-) > > LGTM, but I wonder if it makes any difference at all. Have you compared > the assembled code?
Yes. With 'CPU amdnop' (or intelnop) in effect, padding code uses various multi-byte nop forms. Without it, the single-byte (0x90) form is used. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
