[compiler-rt] [sanitizer-common][rtsan] Mark unsupported tests (PR #213404)
Chris Apple via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 3 07:16:19 PDT 2026
cjappl 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.
https://github.com/llvm/llvm-project/pull/213404
More information about the llvm-commits
mailing list