diff mbox

[FFmpeg-devel] Disable MSA optimization for big endian arch

Message ID B05BEC02968F6E4FB0490058C58B5C8FE0351DAA@PUMAIL01.pu.imgtec.org
State Superseded
Headers show

Commit Message

shivraj.patil@imgtec.com May 15, 2017, 6:01 a.m. UTC 
Hi,
Can anyone review the patch please?

-----Original Message-----
From: Shivraj Patil 
Sent: 24 April 2017 17:33
To: ffmpeg-devel@ffmpeg.org
Cc: Shivraj Patil
Subject: [PATCH] Disable MSA optimization for big endian arch

From: Shivraj Patil <shivraj.patil@imgtec.com>

Signed-off-by: Shivraj Patil <shivraj.patil@imgtec.com>
---
 configure |    4 ++++
 1 file changed, 4 insertions(+)

Comments

Ronald S. Bultje May 15, 2017, noon UTC | #1 
Hi,

On Mon, May 15, 2017 at 2:01 AM, Shivraj Patil <Shivraj.Patil@imgtec.com>
wrote:

> Can anyone review the patch please?
>
[..]

> --- a/configure
> +++ b/configure
> @@ -5357,6 +5357,10 @@ elif enabled mips; then
>      enabled mipsdsp && check_inline_asm_flags mipsdsp '"addu.qb $t0, $t1,
> $t2"' '-mdsp'
>      enabled mipsdspr2 && check_inline_asm_flags mipsdspr2 '"absq_s.qb
> $t0, $t1"' '-mdspr2'
>
> +    if enabled bigendian && enabled msa; then
> +        disable msa
> +    fi
> +
>  elif enabled parisc; then
>
>      if enabled gcc; then
>

<not a configure expert> Looks OK. </not a configure expert>

Ronald
diff mbox

Patch

diff --git a/configure b/configure
index 1e3463c..c63a48a 100755
--- a/configure
+++ b/configure
@@ -5357,6 +5357,10 @@  elif enabled mips; then
     enabled mipsdsp && check_inline_asm_flags mipsdsp '"addu.qb $t0, $t1, $t2"' '-mdsp'
     enabled mipsdspr2 && check_inline_asm_flags mipsdspr2 '"absq_s.qb $t0, $t1"' '-mdspr2'
 
+    if enabled bigendian && enabled msa; then
+        disable msa
+    fi
+
 elif enabled parisc; then
 
     if enabled gcc; then