[MERGED] osmo-trx[master]: buildenv: fix build on systems without SIMD support

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Tom Tsou gerrit-no-reply at lists.osmocom.org
Thu Jun 8 18:32:30 UTC 2017


Tom Tsou has submitted this change and it was merged.

Change subject: buildenv: fix build on systems without SIMD support
......................................................................


buildenv: fix build on systems without SIMD support

HAVE_SSE3 and HAVE_SSE4_1 were never defined if CPU architecture
doesn't match the (86*|x86_64*|amd64*) condition.

Change-Id: I3350b14dbc91e9b388d0b04a0ed22ba27d436313
---
M config/ax_sse.m4
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Tom Tsou: Looks good to me, approved
  Harald Welte: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/config/ax_sse.m4 b/config/ax_sse.m4
index 5c4e321..7d9d7fb 100644
--- a/config/ax_sse.m4
+++ b/config/ax_sse.m4
@@ -40,6 +40,9 @@
 [
   AC_REQUIRE([AC_CANONICAL_HOST])
 
+  AM_CONDITIONAL(HAVE_SSE3, false)
+  AM_CONDITIONAL(HAVE_SSE4_1, false)
+
   case $host_cpu in
     i[[3456]]86*|x86_64*|amd64*)
       AX_CHECK_COMPILE_FLAG(-msse3, ax_cv_support_sse3_ext=yes, [])
@@ -49,7 +52,6 @@
 	AM_CONDITIONAL(HAVE_SSE3, true)
       else
         AC_MSG_WARN([Your compiler does not support sse3 instructions, can you try another compiler?])
-	AM_CONDITIONAL(HAVE_SSE3, false)
       fi
 
       AX_CHECK_COMPILE_FLAG(-msse4.1, ax_cv_support_sse41_ext=yes, [])
@@ -59,7 +61,6 @@
 	AM_CONDITIONAL(HAVE_SSE4_1, true)
       else
         AC_MSG_WARN([Your compiler does not support sse4.1 instructions, can you try another compiler?])
-	AM_CONDITIONAL(HAVE_SSE4_1, false)
       fi
   ;;
   esac

-- 
To view, visit https://gerrit.osmocom.org/2687
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3350b14dbc91e9b388d0b04a0ed22ba27d436313
Gerrit-PatchSet: 2
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Tom Tsou <tom at tsou.cc>



More information about the gerrit-log mailing list