Attention is currently required from: laforge, neels.
fixeria has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmocore/+/35005?usp=email )
Change subject: core: add parity calculation functions for uint{32,16,8}_t
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/libosmocore/+/35005/comment/9c1bc7ee_f8139623
PS1, Line 8:
> I guess it's for the softuart code that Vadim took over from my branch and
> brings it mainline soon.
Yes, Harald's guess is correct. This was actually my initial plan, since I do
need a parity checking logic for the soft-UART. But then I found a simpler
solution and no longer need to use these functions, but I guess somebody might
still find these functions useful?
P.S. I just discovered that both GCC and Clang offer `__builtin_parity`.
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/35005?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I2d6265eac5ce062276a40f4968b129d8702c6117
Gerrit-Change-Number: 35005
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-Attention: neels <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Comment-Date: Tue, 14 Nov 2023 09:59:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: neels <[email protected]>
Comment-In-Reply-To: laforge <[email protected]>
Gerrit-MessageType: comment
