diff mbox

[FFmpeg-devel] avutil/x86inc: don't use movss in VBROADCASTSS macro when src and dst args are the same

Message ID 20170320224149.3808-1-jamrial@gmail.com
State Accepted
Commit d8962ffbd8aaf9485d06eec045b022ba9c39692b
Headers show

Commit Message

James Almer March 20, 2017, 10:41 p.m. UTC 
Signed-off-by: James Almer <jamrial@gmail.com>
---
 libavutil/x86/x86util.asm | 2 ++
 1 file changed, 2 insertions(+)

Comments

Henrik Gramner March 21, 2017, 3:06 p.m. UTC | #1 
x86util, not x86inc. Otherwise LGTM.
James Almer March 21, 2017, 10:16 p.m. UTC | #2 
On 3/21/2017 12:06 PM, Henrik Gramner wrote:
> x86util, not x86inc. Otherwise LGTM.

Fixed and pushed. Thanks
diff mbox

Patch

diff --git a/libavutil/x86/x86util.asm b/libavutil/x86/x86util.asm
index de7d2d11c1..fe9a727e22 100644
--- a/libavutil/x86/x86util.asm
+++ b/libavutil/x86/x86util.asm
@@ -833,7 +833,9 @@ 
 %if cpuflag(avx)
     vbroadcastss %1, %2
 %else ; sse
+%ifnidn %1, %2
     movss        %1, %2
+%endif
     shufps       %1, %1, 0
 %endif
 %endmacro