docs: README accuracy + visible UBSan leg + in-class-tracing regression test (1.7.3)#39
Merged
Merged
Conversation
UBSan already ran bundled in the AddressSanitizer leg; split it into a dedicated unittests-ubsan preset and matrix entry so a UB finding is unambiguous and isolated from memory findings. asan is now ASan (+ LeakSanitizer) only. Signed-off-by: Jo5ta <jo5ta@mail.de>
The GCC 15 COMDAT-safe discovery lifted the old 'cannot trace in a member function defined inside the class body' limitation. Add it as a case to the COMDAT regression test so it stays fixed. Signed-off-by: Jo5ta <jo5ta@mail.de>
- remove the in-class-member-function tracing limitation (now works) - fix the minimal-build flags: CLLTK_DECODER does not exist, the real options are CLLTK_PYTHON_DECODER and CLLTK_CPP_DECODER - describe the decoder output accurately (output.txt by default, .csv optional) instead of a fixed output.csv - document the sanitizer, kernel-module and non-LTO CI steps, the clang-tidy diff mode, and offline cross-endian decoding - version 1.7.3 Signed-off-by: Jo5ta <jo5ta@mail.de>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up (1.7.3): README accuracy + a more visible sanitizer setup, plus a regression test.
README
-DCLLTK_DECODER=OFFis not a real option; the actual flags are-DCLLTK_PYTHON_DECODER=OFF -DCLLTK_CPP_DECODER=OFF.output.txtby default,.csvoptional — instead of a fixedoutput.csv.CI
Test
Verified in the container: the new UBSan leg builds and runs clean, and the in-class regression case passes.