Details
-
Task
-
Status: Open (View Workflow)
-
Major
-
Resolution: Unresolved
Description
To increase the coverage of ASAN, UBSAN and MSAN in our codebase, we should seek to lift the not_*san.inc restrictions from our tests by.
The reasons we limited certain tests from the *SANs include:
Sanitizer bug
This is the primary reason (or at least should be).
However, with the maturity of these tools, up-to-date versions likely have already fixed these bugs.
If more arise, we should expedite upstream corrections rather than put a not_*san.inc lid on them.
"Temporary" kludge to unblock others' work
We should correct the underlying problem.
Nothing is more permanent than the temporary, quoted svoj.
Performance (2× for ASAN, 3× for MSAN)
- The tax is worth the guarantee.
We wouldn't have sanitizer CIs (such as MDBF-1076) at all if this is that much of a concern. - Merging multiple cases already makes a slow kilo-line test.
Conversely, perhaps we should rethink our "big" tests.
Attachments
1.
|
Remove UBSAN limitation from MTR tests |
![]() |
Open | Jimmy Hú |
2.
|
Remove ASAN limitation from MTR tests |
![]() |
Open | Jimmy Hú |
3.
|
Remove MSAN limitation from MTR tests |
![]() |
Open | Unassigned |