[Beignet] [v2 PATCH] remove -ffast-math comiler flag

[Zhigang Gong]

Applied, Thx.

On Tue, Apr 16, 2013 at 02:32:45PM +0800, Lu Guanqun wrote:
> FC16 has the bug https://bugzilla.redhat.com/show_bug.cgi?id=795995
> which prevents us from using our library on FC16.
> 
> Even though it's not our compiler's bug, we don't need fast-math flag here as
> our compiler is not a computation oriented program, so we can remove this flag
> safely.
> 
> Signed-off-by: Lu Guanqun <guanqun.lu at intel.com>
> ---
>  backend/CMakeLists.txt |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/backend/CMakeLists.txt b/backend/CMakeLists.txt
> index 89f3c4a..5498ff0 100644
> --- a/backend/CMakeLists.txt
> +++ b/backend/CMakeLists.txt
> @@ -43,7 +43,7 @@ endif (GBE_DEBUG_MEMORY)
>  set (CMAKE_C_CXX_FLAGS "-fvisibility=hidden")
>  
>  if (COMPILER STREQUAL "GCC")
> -  set (CMAKE_C_CXX_FLAGS "${CMAKE_C_CXX_FLAGS} -funroll-loops -Wstrict-aliasing=2 -fstrict-aliasing -msse2 -msse3 -mssse3 -msse4.1 -ffast-math -fPIC -Wall")
> +  set (CMAKE_C_CXX_FLAGS "${CMAKE_C_CXX_FLAGS} -funroll-loops -Wstrict-aliasing=2 -fstrict-aliasing -msse2 -msse3 -mssse3 -msse4.1 -fPIC -Wall")
>    set (CMAKE_C_CXX_FLAGS "${CMAKE_C_CXX_FLAGS}  ${LLVM_CFLAGS}")
>    set (CMAKE_CXX_FLAGS "${CMAKE_C_CXX_FLAGS}  -Wno-invalid-offsetof -fno-rtti -std=c++0x")
>    set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${GBE_DEBUG_MEMORY_FLAG}")
> @@ -70,7 +70,7 @@ elseif (COMPILER STREQUAL "CLANG")
>    set (CMAKE_C_FLAGS_MINSIZEREL     "-Os -DNDEBUG -DGBE_DEBUG=0")
>    set (CMAKE_C_FLAGS_RELEASE        "-O2 -DNDEBUG -DGBE_DEBUG=0")
>    set (CMAKE_CXX_COMPILER             "clang++")
> -  set (CMAKE_CXX_FLAGS "-fstrict-aliasing -msse2 -ffast-math -fPIC -Wall -Wno-format-security -Wno-invalid-offsetof -std=c++0x")
> +  set (CMAKE_CXX_FLAGS "-fstrict-aliasing -msse2 -fPIC -Wall -Wno-format-security -Wno-invalid-offsetof -std=c++0x")
>    set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${GBE_DEBUG_MEMORY_FLAG}")
>    set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${GBE_COMPILE_UTESTS_FLAG}")
>    set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${VISIBILITY_FLAG}")
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet