Patch Set 1:

> Vadim,
 > 
 > Is sse+avx code just a copy of the sse code? In this case we should
 > move it to an include file and then include into .c files to avoid
 > code duplication.

Both files are almost the same, excluding the SSE_BROADCAST
definition. Great idea with creating an include file, thanks!

WIP...

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I042cc76258df7e4c6c90a73af3d0a6e75999b2b0
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Vadim Yanitskiy <[email protected]>
Gerrit-Reviewer: Alexander Chemeris <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Tom Tsou <[email protected]>
Gerrit-Reviewer: Vadim Yanitskiy <[email protected]>
Gerrit-Reviewer: tnt <[email protected]>
Gerrit-HasComments: No

Reply via email to