Skip to content

Conversation

@lzaoral
Copy link
Member

@lzaoral lzaoral commented Dec 12, 2023

Resolves: #150

@lzaoral lzaoral requested a review from kdudka December 12, 2023 12:55
@lzaoral lzaoral self-assigned this Dec 12, 2023
d->addMsgFilter("", "at least [0-9][0-9]* times.$");

// squash UBSAN reports that only differ in the address
d->addMsgFilter("UBSAN_WARNING", "0x[[:xdigit:]]+");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be safer to use a more specific regex, e.g. prepend the address prefix. But this can be tweaked later as the support for ubsan is still experimental.

@lzaoral lzaoral force-pushed the deduplicate-ubsan-address branch from efeee29 to 941ad1a Compare January 3, 2024 09:02
@lzaoral lzaoral merged commit 941ad1a into csutils:main Jan 3, 2024
@lzaoral lzaoral deleted the deduplicate-ubsan-address branch January 3, 2024 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Do better at UBSAN warning deduplication

2 participants