Build: Make it possible to run MSAN in GN
MSAN has false positives due to b/234876100, but it works correctly for
most of the Pigweed codebase. This change makes it possible to use the
top-level `msan` target, but does not add msan to the default build.
Previously, the target was present, but was always a no-op.
Change-Id: I6811bb0f24f879f73c1bbf6afa3ffbf928f6d963
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/167112
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Armando Montanez <amontanez@google.com>
Pigweed-Auto-Submit: Wyatt Hepler <hepler@google.com>
diff --git a/BUILD.gn b/BUILD.gn
index a24411a..9c26edd 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -356,13 +356,11 @@
}
}
-# TODO(b/234876100): msan will not work until the C++ standard library included
-# in the sysroot has a variant built with msan.
+# TODO(b/234876100): msan will not work without false positives until the C++
+# standard library included in the sysroot has a variant built with msan.
group("msan") {
# TODO(b/259695498): msan doesn't work on macOS yet.
- if (host_os != "win" && host_os != "mac" && host_os != "linux") {
- deps = [ ":pw_module_tests.run($dir_pigweed/targets/host:host_clang_msan)" ]
- }
+ deps = [ ":pw_module_tests.run($dir_pigweed/targets/host:host_clang_msan)" ]
}
group("tsan") {
