From 547b93c9c2c58cfbeb4064ea0e89e92da324d898 Mon Sep 17 00:00:00 2001
From: Matthew Gregan <kinetik@flim.org>
Date: Sat, 19 Jun 2010 11:42:11 +1200
Subject: [PATCH] Backed out changeset d268e54fbfcf (bug 572034)

---
 gfx/ycbcr/Makefile.in               |  5 ---
 gfx/ycbcr/README                    |  1 -
 gfx/ycbcr/bug572034_mac_64bit.patch | 66 -----------------------------
 gfx/ycbcr/update.sh                 |  1 -
 gfx/ycbcr/yuv_row_linux.cpp         | 14 +++---
 5 files changed, 7 insertions(+), 80 deletions(-)
 delete mode 100644 gfx/ycbcr/bug572034_mac_64bit.patch

diff --git a/gfx/ycbcr/Makefile.in b/gfx/ycbcr/Makefile.in
index c030f6e224c..30b01982e6e 100644
--- a/gfx/ycbcr/Makefile.in
+++ b/gfx/ycbcr/Makefile.in
@@ -30,13 +30,8 @@ CPPSRCS += yuv_row_linux.cpp \
            $(NULL)
 else
 ifeq ($(OS_ARCH),Darwin)
-ifeq ($(OS_TEST),x86_64)
-CPPSRCS += yuv_row_linux.cpp \
-           $(NULL)
-else
 CPPSRCS += yuv_row_mac.cpp \
            $(NULL)
-endif
 else
 CPPSRCS += yuv_row_other.cpp \
            $(NULL)
diff --git a/gfx/ycbcr/README b/gfx/ycbcr/README
index 4c248068c7e..81433a7268c 100644
--- a/gfx/ycbcr/README
+++ b/gfx/ycbcr/README
@@ -19,4 +19,3 @@ export.patch: Fix export for building on comm-central
 win64_mac64.patch: Fallback to C implementation on Windows and Mac OS X 64 bit
 yv24.patch: Adds YCbCr 4:4:4 support
 row_c_fix.patch: Fix broken C fallback code (See bug 561385).
-bug572034_mac_64bit.patch: Fix x86_64 linux code so it works on OS X.
diff --git a/gfx/ycbcr/bug572034_mac_64bit.patch b/gfx/ycbcr/bug572034_mac_64bit.patch
deleted file mode 100644
index 413cc8fd8ae..00000000000
--- a/gfx/ycbcr/bug572034_mac_64bit.patch
+++ /dev/null
@@ -1,66 +0,0 @@
-diff --git a/gfx/ycbcr/yuv_row_linux.cpp b/gfx/ycbcr/yuv_row_linux.cpp
---- a/gfx/ycbcr/yuv_row_linux.cpp
-+++ b/gfx/ycbcr/yuv_row_linux.cpp
-@@ -250,18 +250,18 @@ MMX_ALIGNED(int16 kCoefficientsRgbY[768]
- 
- // AMD64 ABI uses register paremters.
- void FastConvertYUVToRGB32Row(const uint8* y_buf,  // rdi
-                               const uint8* u_buf,  // rsi
-                               const uint8* v_buf,  // rdx
-                               uint8* rgb_buf,      // rcx
-                               int width) {         // r8
-   asm(
--  "jmp    convertend\n"
--"convertloop:"
-+  "jmp    Lconvertend\n"
-+"Lconvertloop:"
-   "movzb  (%1),%%r10\n"
-   "add    $0x1,%1\n"
-   "movzb  (%2),%%r11\n"
-   "add    $0x1,%2\n"
-   "movq   2048(%5,%%r10,8),%%xmm0\n"
-   "movzb  (%0),%%r10\n"
-   "movq   4096(%5,%%r11,8),%%xmm1\n"
-   "movzb  0x1(%0),%%r11\n"
-@@ -271,36 +271,36 @@ void FastConvertYUVToRGB32Row(const uint
-   "movq   (%5,%%r11,8),%%xmm3\n"
-   "paddsw %%xmm0,%%xmm2\n"
-   "paddsw %%xmm0,%%xmm3\n"
-   "shufps $0x44,%%xmm3,%%xmm2\n"
-   "psraw  $0x6,%%xmm2\n"
-   "packuswb %%xmm2,%%xmm2\n"
-   "movq   %%xmm2,0x0(%3)\n"
-   "add    $0x8,%3\n"
--"convertend:"
-+"Lconvertend:"
-   "sub    $0x2,%4\n"
--  "jns    convertloop\n"
-+  "jns    Lconvertloop\n"
- 
--"convertnext:"
-+"Lconvertnext:"
-   "add    $0x1,%4\n"
--  "js     convertdone\n"
-+  "js     Lconvertdone\n"
- 
-   "movzb  (%1),%%r10\n"
-   "movq   2048(%5,%%r10,8),%%xmm0\n"
-   "movzb  (%2),%%r10\n"
-   "movq   4096(%5,%%r10,8),%%xmm1\n"
-   "paddsw %%xmm1,%%xmm0\n"
-   "movzb  (%0),%%r10\n"
-   "movq   (%5,%%r10,8),%%xmm1\n"
-   "paddsw %%xmm0,%%xmm1\n"
-   "psraw  $0x6,%%xmm1\n"
-   "packuswb %%xmm1,%%xmm1\n"
-   "movd   %%xmm1,0x0(%3)\n"
--"convertdone:"
-+"Lconvertdone:"
-   :
-   : "r"(y_buf),  // %0
-     "r"(u_buf),  // %1
-     "r"(v_buf),  // %2
-     "r"(rgb_buf),  // %3
-     "r"(width),  // %4
-     "r" (kCoefficientsRgbY)  // %5
-   : "memory", "r10", "r11", "xmm0", "xmm1", "xmm2", "xmm3"
diff --git a/gfx/ycbcr/update.sh b/gfx/ycbcr/update.sh
index b46e1996b4b..2836d3cd43c 100644
--- a/gfx/ycbcr/update.sh
+++ b/gfx/ycbcr/update.sh
@@ -13,4 +13,3 @@ patch -p3 <export.patch
 patch -p3 <win64_mac64.patch
 patch -p3 <yv24.patch
 patch -p3 <row_c_fix.patch
-patch -p3 <bug572034_mac_64bit.patch
diff --git a/gfx/ycbcr/yuv_row_linux.cpp b/gfx/ycbcr/yuv_row_linux.cpp
index 515a4557c0f..455dd7bc29f 100644
--- a/gfx/ycbcr/yuv_row_linux.cpp
+++ b/gfx/ycbcr/yuv_row_linux.cpp
@@ -255,8 +255,8 @@ void FastConvertYUVToRGB32Row(const uint8* y_buf,  // rdi
                               uint8* rgb_buf,      // rcx
                               int width) {         // r8
   asm(
-  "jmp    Lconvertend\n"
-"Lconvertloop:"
+  "jmp    convertend\n"
+"convertloop:"
   "movzb  (%1),%%r10\n"
   "add    $0x1,%1\n"
   "movzb  (%2),%%r11\n"
@@ -276,13 +276,13 @@ void FastConvertYUVToRGB32Row(const uint8* y_buf,  // rdi
   "packuswb %%xmm2,%%xmm2\n"
   "movq   %%xmm2,0x0(%3)\n"
   "add    $0x8,%3\n"
-"Lconvertend:"
+"convertend:"
   "sub    $0x2,%4\n"
-  "jns    Lconvertloop\n"
+  "jns    convertloop\n"
 
-"Lconvertnext:"
+"convertnext:"
   "add    $0x1,%4\n"
-  "js     Lconvertdone\n"
+  "js     convertdone\n"
 
   "movzb  (%1),%%r10\n"
   "movq   2048(%5,%%r10,8),%%xmm0\n"
@@ -295,7 +295,7 @@ void FastConvertYUVToRGB32Row(const uint8* y_buf,  // rdi
   "psraw  $0x6,%%xmm1\n"
   "packuswb %%xmm1,%%xmm1\n"
   "movd   %%xmm1,0x0(%3)\n"
-"Lconvertdone:"
+"convertdone:"
   :
   : "r"(y_buf),  // %0
     "r"(u_buf),  // %1