Bug#873608: patch

[Paul Tagliamonte]

Whoops, I only just read the comment about this not being standard in
armhf. I misread that and also assumed it was standard (likely because,
as a Coretex-A* user, I always had it), so this patch will likely be
inappropriate (or incomplete). Maybe this patch plus an AND on an
ENABLE_NEON or something, which is disabled by default in the packaging.

  Paul


On Tue, Aug 29, 2017 at 04:10:30PM -0400, Paul Tagliamonte wrote:
> tags 873608 + patch
> thanks
> 
> Attached is a patch that will enable neon if arm_neon.h is present. I
> didn't upstream this or anything, I figure you have a better
> relationship with them, but this ought to fix the ftbfs.
> 
> I'm running a test build now, but it's past the old ftbfs point
> 
>   Paul
> 
> -- 



-- 

Bug#873608: patch

[Paul Tagliamonte]

tags 873608 + patch
thanks

Attached is a patch that will enable neon if arm_neon.h is present. I
didn't upstream this or anything, I figure you have a better
relationship with them, but this ought to fix the ftbfs.

I'm running a test build now, but it's past the old ftbfs point

  Paul

-- 
Description: When building for armhf, enable NEON
 NEON is part of the armhf baseline, so this will always be enabled on
 armhf.
Author: Paul Tagliamonte 
Bug-Debian: https://bugs.debian.org/873608
Origin: vendor
Last-Update: 2017-08-29

--- uhd-3.10.2.0.orig/host/lib/convert/CMakeLists.txt
+++ uhd-3.10.2.0/host/lib/convert/CMakeLists.txt
@@ -67,6 +67,8 @@ IF(HAVE_ARM_NEON_H AND (${CMAKE_SIZEOF_V
 ${CMAKE_CURRENT_SOURCE_DIR}/convert_with_neon.cpp
 ${CMAKE_CURRENT_SOURCE_DIR}/convert_neon.S
 )
+
+SET ( CMAKE_CXX_FLAGS "-mfpu=neon" )
 ENDIF()