On 09/05/16 16:46, Vinson Lee wrote:
This patch fixes this build error.

   CXX      rasterizer/memory/libswrAVX_la-ClearTile.lo
In file included from rasterizer/memory/ClearTile.cpp:34:0:
./rasterizer/memory/Convert.h: In function ‘uint16_t Convert32To16Float(float)’:
./rasterizer/memory/Convert.h:170:9: error: ‘__builtin_isnan’ is not a member 
of ‘std’
      if (std::isnan(val))
          ^
./rasterizer/memory/Convert.h:170:9: note: suggested alternative:
<built-in>: note:   ‘__builtin_isnan’
./rasterizer/memory/Convert.h:176:14: error: ‘__builtin_isinf_sign’ is not a 
member of ‘std’
      else if (std::isinf(val))
               ^
./rasterizer/memory/Convert.h:176:14: note: suggested alternative:
<built-in>: note:   ‘__builtin_isinf_sign’

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95180
Signed-off-by: Vinson Lee <[email protected]>
---
  src/gallium/drivers/swr/rasterizer/memory/Convert.h | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/src/gallium/drivers/swr/rasterizer/memory/Convert.h 
b/src/gallium/drivers/swr/rasterizer/memory/Convert.h
index 4c4642f..42b973c 100644
--- a/src/gallium/drivers/swr/rasterizer/memory/Convert.h
+++ b/src/gallium/drivers/swr/rasterizer/memory/Convert.h
@@ -32,6 +32,8 @@
  #pragma warning(disable: 4723)
  #endif

+#include <cmath>
+
  //////////////////////////////////////////////////////////////////////////
  /// @brief Convert an IEEE 754 16-bit float to an 32-bit single precision
  ///        float


Looks good to me.

Reviewed-by: Jose Fonseca <[email protected]>
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to