This silences lots of CMake warnings about adding dependencies on
non-existing targets.
Details
Details
- Reviewers
samsonov
Diff Detail
Diff Detail
Event Timeline
Comment Actions
What CMake invocation produces such errors?
Anyway, this patch brings up a valid issue, but I think we should better determine which sanitizers are actually going to be built in the top-level CMakeLists.txt (or move this logic to some helper module). Let me know if you will update the patch, otherwise I'll make this change myself.
Comment Actions
On Windows, only ASan is built. However, we add the test suites for profile, tsan, and msan, all of which reference non-existing targets. This prevents us from doing that. If you want to hoist the logic for these decisions out of lib/CMakeLists, go for it, I'll close this.