[PATCH] D34644: Only test sanitizers that are built when COMPILER_RT_SANITIZERS_TO_BUILD is used

[Peter Collingbourne via Phabricator via llvm-commits]

pcc accepted this revision.
pcc added a comment.

In https://reviews.llvm.org/D34644#792470, @fjricci wrote:

> As far as I'm aware, the feature exists both to save time building and testing sanitizers you don't need, and to get around differing build requirements across the sanitizers (it's possible, especially when cross-compiling, that some sanitizers build in a given configuration and others do not).


You can achieve that though by choosing not to build the sanitizer runtimes that you don't want when you run "make" or "ninja" or whatever.

But anyway, this seems fine to me. LGTM.


https://reviews.llvm.org/D34644