Change in libosmocore[master]: configure.ac: fix: do not define HAVE_NEON unconditionally

2020-08-06 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19545 )

Change subject: configure.ac: fix: do not define HAVE_NEON unconditionally
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/19545
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I761a7afaeda9d232ac26edff47949e911f8f1f0c
Gerrit-Change-Number: 19545
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Thu, 06 Aug 2020 21:12:13 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in libosmocore[master]: configure.ac: fix: do not define HAVE_NEON unconditionally

2020-08-06 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19545 )

Change subject: configure.ac: fix: do not define HAVE_NEON unconditionally
..

configure.ac: fix: do not define HAVE_NEON unconditionally

Change-Id: I761a7afaeda9d232ac26edff47949e911f8f1f0c
---
M configure.ac
1 file changed, 0 insertions(+), 2 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/configure.ac b/configure.ac
index cc8d3a7..d249be9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -384,8 +384,6 @@
[Enable ARM NEON instructions support [default=no]]
)],
[neon=$enableval], [neon="no"])
-AC_DEFINE(HAVE_NEON,,
-[Support ARM NEON instructions])
 AM_CONDITIONAL(HAVE_NEON, [test "x$neon" != "xno"])



--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/19545
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I761a7afaeda9d232ac26edff47949e911f8f1f0c
Gerrit-Change-Number: 19545
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged


Change in libosmocore[master]: configure.ac: fix: do not define HAVE_NEON unconditionally

2020-08-06 Thread fixeria
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19545 )


Change subject: configure.ac: fix: do not define HAVE_NEON unconditionally
..

configure.ac: fix: do not define HAVE_NEON unconditionally

Change-Id: I761a7afaeda9d232ac26edff47949e911f8f1f0c
---
M configure.ac
1 file changed, 0 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/45/19545/1

diff --git a/configure.ac b/configure.ac
index cc8d3a7..d249be9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -384,8 +384,6 @@
[Enable ARM NEON instructions support [default=no]]
)],
[neon=$enableval], [neon="no"])
-AC_DEFINE(HAVE_NEON,,
-[Support ARM NEON instructions])
 AM_CONDITIONAL(HAVE_NEON, [test "x$neon" != "xno"])



--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/19545
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I761a7afaeda9d232ac26edff47949e911f8f1f0c
Gerrit-Change-Number: 19545
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange