convert: move priorities to implementation, different for arm

3 files changed, 19 insertions, 20 deletions

diff --git a/host/include/uhd/convert.hpp b/host/include/uhd/convert.hpp
index 6cc729819..f906ff0e9 100644
--- a/host/include/uhd/convert.hpp
+++ b/host/include/uhd/convert.hpp
@@ -50,21 +50,8 @@ namespace uhd{ namespace convert{
     //! Conversion factory function typedef
     typedef boost::function<converter::sptr(void)> function_type;
 
-    /*!
-     * Describe the priority of a converter function.
-     * A higher priority function takes precedence.
-     * The general case function are the lowest.
-     * Next comes the liborc implementations.
-     * Custom intrinsics implementations are highest.
-     */
-    enum priority_type{
-        PRIORITY_GENERAL = 0,
-        PRIORITY_LIBORC = 1,
-        PRIORITY_SIMD = 2,
-        PRIORITY_TABLE = 3,
-        PRIORITY_CUSTOM = 4,
-        PRIORITY_EMPTY = -1,
-    };
+    //! Priority of conversion routines
+    typedef int priority_type;
 
     //! Identify a conversion routine in the registry
     struct id_type : boost::equality_comparable<id_type>{
diff --git a/host/lib/convert/CMakeLists.txt b/host/lib/convert/CMakeLists.txt
index 1fe92d93a..98907dc29 100644
--- a/host/lib/convert/CMakeLists.txt
+++ b/host/lib/convert/CMakeLists.txt
@@ -91,11 +91,7 @@ IF(CMAKE_COMPILER_IS_GNUCXX)
     UNSET(CMAKE_REQUIRED_FLAGS)
 ENDIF(CMAKE_COMPILER_IS_GNUCXX)
 
-IF(HAVE_ARM_NEON_H AND ENABLE_ORC)
-    #prefer orc support, its faster than the current intrinsic implementations
-    MESSAGE(STATUS "Enabled conversion support with ORC.")
-ELSEIF(HAVE_ARM_NEON_H)
-    MESSAGE(STATUS "Enabled conversion support with NEON intrinsics.")
+IF(HAVE_ARM_NEON_H)
     SET_SOURCE_FILES_PROPERTIES(
         ${CMAKE_CURRENT_SOURCE_DIR}/convert_with_neon.cpp
         PROPERTIES COMPILE_FLAGS "${NEON_FLAGS}"
diff --git a/host/lib/convert/convert_common.hpp b/host/lib/convert/convert_common.hpp
index f963e29ee..cc287114a 100644
--- a/host/lib/convert/convert_common.hpp
+++ b/host/lib/convert/convert_common.hpp
@@ -46,6 +46,22 @@
     _DECLARE_CONVERTER(__convert_##in_form##_##num_in##_##out_form##_##num_out##_##prio, in_form, num_in, out_form, num_out, prio)
 
 /***********************************************************************
+ * Setup priorities
+ **********************************************************************/
+static const int PRIORITY_GENERAL = 0;
+static const int PRIORITY_EMPTY = -1;
+
+#ifdef __ARM_NEON__
+static const int PRIORITY_LIBORC = 3;
+static const int PRIORITY_SIMD = 1; //neon conversions could be implemented better, orc wins
+static const int PRIORITY_TABLE = 2; //tables require large cache, so they are slower on arm
+#else
+static const int PRIORITY_LIBORC = 1;
+static const int PRIORITY_SIMD = 2;
+static const int PRIORITY_TABLE = 3;
+#endif
+
+/***********************************************************************
  * Typedefs
  **********************************************************************/
 typedef std::complex<double>         fc64_t;