[compiler-rt] [sanitizer-common][rtsan] Mark unsupported tests (PR #213404)
Vitaly Buka via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 3 21:58:53 PDT 2026
vitalybuka wrote:
> > This LGTM, but let's wait for feedback from Vitaly and Florian before merging
>
> Sounds good. The main feedback I want for this is the OVERALL feedback from that other review, that is "should rtsan be a part of the sanitizer common tests?"
>
> My opinion is yes, it gives us better coverage and makes us more conformant with standard expectations. It pointed out a couple bugs that we have had in there for years, and it may have made our life easier implementing some of the standard stuff originally. I also believe rtsan to be the only sanitizer not in this group right now.
>
> The downside is just more runtime, adding another sanitizer means more test time for any CI using this suite.
What is fraction % of tests you had to mark UNSUPPORTED?
https://github.com/llvm/llvm-project/pull/213404
More information about the llvm-commits
mailing list