Patch Set 1: (1 comment)
https://gerrit.osmocom.org/#/c/2102/1/config/ax_ext.m4 File config/ax_ext.m4: Line 51: ax_cv_have_mmx_ext=no why not either set all of them to yes, or remove the sse2/sse/avx bits below? The code after your patch might be easily misunderstood to set HAVE_SSE4_2 depending on compiler support, but actually it is not and all the related autoconf macros below are dead code, right? -- To view, visit https://gerrit.osmocom.org/2102 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic913aa13c23c348ae62e78c9dfd6ed8b0a62798c Gerrit-PatchSet: 1 Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Owner: dexter <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: Yes
