From 7d5303a083c93f1a04b09aaa6fd35481315b043e Mon Sep 17 00:00:00 2001
From: Gael Guennebaud <g.gael@free.fr>
Date: Fri, 16 Dec 2016 09:25:14 +0100
Subject: Partly revert changeset 642dddcce29269f266d35e34d34ee83d99a7c116 ,
 just in case the x87 issue popup again

---
 test/geo_alignedbox.cpp | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

(limited to 'test/geo_alignedbox.cpp')

diff --git a/test/geo_alignedbox.cpp b/test/geo_alignedbox.cpp
index eca292f4c..223ff5eea 100644
--- a/test/geo_alignedbox.cpp
+++ b/test/geo_alignedbox.cpp
@@ -15,9 +15,17 @@
 #include<iostream>
 using namespace std;
 
-// TODO not sure if this is actually still necessary anywhere ...
+// NOTE the following workaround was needed on some 32 bits builds to kill extra precision of x87 registers.
+// It seems that it os not needed anymore, but let's keep it here, just in case...
+
 template<typename T> EIGEN_DONT_INLINE
-void kill_extra_precision(T& ) {  }
+void kill_extra_precision(T& /* x */) {
+  // This one worked but triggered a warning:
+  /* eigen_assert((void*)(&x) != (void*)0); */
+  // An alternative could be:
+  /* volatile T tmp = x; */
+  /* x = tmp; */
+}
 
 
 template<typename BoxType> void alignedbox(const BoxType& _box)
-- 
cgit v1.2.3