Update UBSan suppressions file with suppressions needed for clang 12 (current trunk).
tests: Update UBSan suppressions file with suppressions needed for clang 12 (current trunk) #20137
pull practicalswift wants to merge 1 commits into bitcoin:master from practicalswift:clang-12-ubsan-suppressions changing 1 files +26 −0-
practicalswift commented at 8:26 PM on October 12, 2020: contributor
-
tests: Update UBSan suppressions file with suppressions needed for clang 12 (current trunk) 3227229861
- DrahtBot added the label Tests on Oct 12, 2020
-
fanquake commented at 3:38 AM on October 15, 2020: member
Seems ok to me if this is useful when testing withj sanitizers. Although given it's commented out, it's just the difference of applying your own patches vs un-commenting the lines pre-build? @MarcoFalke can have the final say.
-
practicalswift commented at 6:13 AM on October 15, 2020: contributor
Although given it's commented out, it's just the difference of applying your own patches vs un-commenting the lines pre-build?
The reason it is commented out is that older versions of UBSan errors on suppressions it cannot parse. We'll need this in
masteruncommented eventually (when upgrading to Clang 12).For me personally I already have the patch in my repo, but I'm hopefully not the only one building Bitcoin Core with the latest version of Clang (to get early notification of compilation issues/compiler bugs, to get the best libFuzzer and diagnostics goodies, etc.) we support so I thought it would be helpful to upstream it commented out for now :)
- practicalswift closed this on Nov 11, 2020
- practicalswift deleted the branch on Apr 10, 2021
- DrahtBot locked this on Aug 18, 2022
Contributors
Labels