Attachment 783 Details for Bug 1411 – Patch

This bugzilla service is closed. All entries have been migrated to https://gitlab.com/libeigen/eigen

[patch] Patch

eigen.patch (text/plain), 1.62 KB, created by Shuntaro Yamazaki on 2017-03-28 03:50:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Shuntaro Yamazaki

Created: 2017-03-28 03:50:51 UTC

Size: 1.62 KB

patch

obsolete

>diff -r 525f03452c37 Eigen/src/Geometry/arch/Geometry_SSE.h
>--- a/Eigen/src/Geometry/arch/Geometry_SSE.h	Fri Mar 17 17:33:15 2017 +0100
>+++ b/Eigen/src/Geometry/arch/Geometry_SSE.h	Tue Mar 28 12:34:02 2017 +0900
>@@ -97,9 +97,9 @@
>   t2 = psub(pmul(a_zz, b_xy), pmul(a_xx, b_zw));
> #ifdef EIGEN_VECTORIZE_SSE3
>   EIGEN_UNUSED_VARIABLE(mask)
>-  pstore(&res.x(), _mm_addsub_pd(t1, preverse(t2)));
>+  pstoret<double,Packet2d,Alignment>(&res.x(), _mm_addsub_pd(t1, preverse(t2)));
> #else
>-  pstore(&res.x(), padd(t1, pxor(mask,preverse(t2))));
>+  pstoret<double,Packet2d,Alignment>(&res.x(), padd(t1, pxor(mask,preverse(t2))));
> #endif
>   
>   /*
>@@ -111,9 +111,9 @@
>   t2 = padd(pmul(a_zz, b_zw), pmul(a_xx, b_xy));
> #ifdef EIGEN_VECTORIZE_SSE3
>   EIGEN_UNUSED_VARIABLE(mask)
>-  pstore(&res.z(), preverse(_mm_addsub_pd(preverse(t1), t2)));
>+  pstoret<double,Packet2d,Alignment>(&res.z(), preverse(_mm_addsub_pd(preverse(t1), t2)));
> #else
>-  pstore(&res.z(), psub(t1, pxor(mask,preverse(t2))));
>+  pstoret<double,Packet2d,Alignment>(&res.z(), psub(t1, pxor(mask,preverse(t2))));
> #endif
> 
>   return res;
>@@ -128,8 +128,8 @@
>     Quaternion<double> res;
>     const __m128d mask0 = _mm_setr_pd(-0.,-0.);
>     const __m128d mask2 = _mm_setr_pd(-0.,0.);
>-    pstore(&res.x(), _mm_xor_pd(mask0, q.coeffs().template packet<Alignment>(0)));
>-    pstore(&res.z(), _mm_xor_pd(mask2, q.coeffs().template packet<Alignment>(2)));
>+    pstoret<double,Packet2d,Alignment>(&res.x(), _mm_xor_pd(mask0, q.coeffs().template packet<Alignment>(0)));
>+    pstoret<double,Packet2d,Alignment>(&res.z(), _mm_xor_pd(mask2, q.coeffs().template packet<Alignment>(2)));
>     return res;
>   }
> };

Actions: View | Diff

Attachments on bug 1411: 783