laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/14682 )
Change subject: tell ubsan to ignore SUN_LEN ...................................................................... Patch Set 3: Code-Review+2 (1 comment) https://gerrit.osmocom.org/#/c/14682/3//COMMIT_MSG Commit Message: https://gerrit.osmocom.org/#/c/14682/3//COMMIT_MSG@15 PS3, Line 15: I sincerly hope that this explanation provides to be sufficient, If not I'd be willing to show up at : the next llvm dev meeting to provide quotes from actual sanitizer developers to back up these claims. The commit log may appear excessively verbose to you, but that's only because it appears you have much more insight into [modern?] compiler/toolchain topics than most other Osmocom developers, at the very leat myself. So it really helps, thanks! -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14682 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I0ff445072f1b46390c9f70b21d61c789e39358d5 Gerrit-Change-Number: 14682 Gerrit-PatchSet: 3 Gerrit-Owner: Hoernchen <[email protected]> Gerrit-Reviewer: Hoernchen <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Comment-Date: Sat, 20 Jul 2019 06:08:34 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
