summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOliver Chang <ochang@chromium.org>2016-03-10 11:06:16 -0800
committerOliver Chang <ochang@chromium.org>2016-03-10 11:06:16 -0800
commit2852ce07c4e5b1053420cbf375e098e09faf679c (patch)
treec683d458b5113f75c6776abc01fb18a647030e7a
parentf04089059b140ad50ed386b509ab200ce84b66e5 (diff)
downloadpdfium-2852ce07c4e5b1053420cbf375e098e09faf679c.tar.xz
openjpeg: Don't build with SSE instrinsics for 32-bit x86.
Fixes 32-bit clang-cl build. These intrinsics require addresses with 16-byte alignment, which doesn't cause issues for 64-bit builds because of allocator alignment. MSVC doesn't recognise the __SSE__, __SSE2__ defines, which is why this problem never surfaced before. R=tsepez@chromium.org BUG=pdfium:433 Review URL: https://codereview.chromium.org/1783813003 .
-rw-r--r--third_party/libopenjpeg20/0012-mct_sse.patch60
-rw-r--r--third_party/libopenjpeg20/README.pdfium1
-rw-r--r--third_party/libopenjpeg20/mct.c17
3 files changed, 71 insertions, 7 deletions
diff --git a/third_party/libopenjpeg20/0012-mct_sse.patch b/third_party/libopenjpeg20/0012-mct_sse.patch
new file mode 100644
index 0000000000..bf08276764
--- /dev/null
+++ b/third_party/libopenjpeg20/0012-mct_sse.patch
@@ -0,0 +1,60 @@
+diff --git a/third_party/libopenjpeg20/mct.c b/third_party/libopenjpeg20/mct.c
+index 8b0276f..7731c15 100644
+--- a/third_party/libopenjpeg20/mct.c
++++ b/third_party/libopenjpeg20/mct.c
+@@ -37,13 +37,16 @@
+ * POSSIBILITY OF SUCH DAMAGE.
+ */
+
+-#ifdef __SSE__
++#if defined(__SSE__) && !defined(_M_IX86) && !defined(__i386)
++#define USE_SSE
+ #include <xmmintrin.h>
+ #endif
+-#ifdef __SSE2__
++#if defined(__SSE2__) && !defined(_M_IX86) && !defined(__i386)
++#define USE_SSE2
+ #include <emmintrin.h>
+ #endif
+-#ifdef __SSE4_1__
++#if defined(__SSE4_1__) && !defined(_M_IX86) && !defined(__i386)
++#define USE_SSE4
+ #include <smmintrin.h>
+ #endif
+
+@@ -72,7 +75,7 @@ const OPJ_FLOAT64 * opj_mct_get_mct_norms_real ()
+ /* <summary> */
+ /* Forward reversible MCT. */
+ /* </summary> */
+-#ifdef __SSE2__
++#ifdef USE_SSE2
+ void opj_mct_encode(
+ OPJ_INT32* restrict c0,
+ OPJ_INT32* restrict c1,
+@@ -137,7 +140,7 @@ void opj_mct_encode(
+ /* <summary> */
+ /* Inverse reversible MCT. */
+ /* </summary> */
+-#ifdef __SSE2__
++#ifdef USE_SSE2
+ void opj_mct_decode(
+ OPJ_INT32* restrict c0,
+ OPJ_INT32* restrict c1,
+@@ -204,7 +207,7 @@ OPJ_FLOAT64 opj_mct_getnorm(OPJ_UINT32 compno) {
+ /* <summary> */
+ /* Forward irreversible MCT. */
+ /* </summary> */
+-#ifdef __SSE4_1__
++#ifdef USE_SSE4
+ void opj_mct_encode_real(
+ OPJ_INT32* restrict c0,
+ OPJ_INT32* restrict c1,
+@@ -377,7 +380,7 @@ void opj_mct_decode_real(
+ OPJ_UINT32 n)
+ {
+ OPJ_UINT32 i;
+-#ifdef __SSE__
++#ifdef USE_SSE
+ __m128 vrv, vgu, vgv, vbu;
+ vrv = _mm_set1_ps(1.402f);
+ vgu = _mm_set1_ps(0.34413f);
diff --git a/third_party/libopenjpeg20/README.pdfium b/third_party/libopenjpeg20/README.pdfium
index 0a60ed7691..cec6064ea2 100644
--- a/third_party/libopenjpeg20/README.pdfium
+++ b/third_party/libopenjpeg20/README.pdfium
@@ -21,4 +21,5 @@ Local Modifications:
0009-opj_pi_next.patch: Fix potential bad precno value in opj_pi_next* functions.
0010-pi_update_decode_poc.patch: Set proper upper bound for an array in opj_pi_update_decode_poc().
0011-j2k_update_image_data.patch: Prevent bad signed -> unsigned casting.
+0012-mct_sse.patch: Don't use SSE intrinsics in 32-bit builds.
TODO(thestig): List all the other patches.
diff --git a/third_party/libopenjpeg20/mct.c b/third_party/libopenjpeg20/mct.c
index 8b0276f32d..7731c15ddb 100644
--- a/third_party/libopenjpeg20/mct.c
+++ b/third_party/libopenjpeg20/mct.c
@@ -37,13 +37,16 @@
* POSSIBILITY OF SUCH DAMAGE.
*/
-#ifdef __SSE__
+#if defined(__SSE__) && !defined(_M_IX86) && !defined(__i386)
+#define USE_SSE
#include <xmmintrin.h>
#endif
-#ifdef __SSE2__
+#if defined(__SSE2__) && !defined(_M_IX86) && !defined(__i386)
+#define USE_SSE2
#include <emmintrin.h>
#endif
-#ifdef __SSE4_1__
+#if defined(__SSE4_1__) && !defined(_M_IX86) && !defined(__i386)
+#define USE_SSE4
#include <smmintrin.h>
#endif
@@ -72,7 +75,7 @@ const OPJ_FLOAT64 * opj_mct_get_mct_norms_real ()
/* <summary> */
/* Forward reversible MCT. */
/* </summary> */
-#ifdef __SSE2__
+#ifdef USE_SSE2
void opj_mct_encode(
OPJ_INT32* restrict c0,
OPJ_INT32* restrict c1,
@@ -137,7 +140,7 @@ void opj_mct_encode(
/* <summary> */
/* Inverse reversible MCT. */
/* </summary> */
-#ifdef __SSE2__
+#ifdef USE_SSE2
void opj_mct_decode(
OPJ_INT32* restrict c0,
OPJ_INT32* restrict c1,
@@ -204,7 +207,7 @@ OPJ_FLOAT64 opj_mct_getnorm(OPJ_UINT32 compno) {
/* <summary> */
/* Forward irreversible MCT. */
/* </summary> */
-#ifdef __SSE4_1__
+#ifdef USE_SSE4
void opj_mct_encode_real(
OPJ_INT32* restrict c0,
OPJ_INT32* restrict c1,
@@ -377,7 +380,7 @@ void opj_mct_decode_real(
OPJ_UINT32 n)
{
OPJ_UINT32 i;
-#ifdef __SSE__
+#ifdef USE_SSE
__m128 vrv, vgu, vgv, vbu;
vrv = _mm_set1_ps(1.402f);
vgu = _mm_set1_ps(0.34413f);