[PATCH] D71131: gn build: Change scudo's list of supported platforms to a whitelist.
Nico Weber via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 6 11:18:32 PST 2019
thakis accepted this revision.
thakis added a comment.
This revision is now accepted and ready to land.
Thanks! lg.
I'd probably find it easier to understand, or at least more self-consistent, if this file had an assert(is_linux || is_fuchsia), and llvm/utils/gn/secondary/BUILD.gn had just the dep on //compiler-rt it already had, and the supported_toolchain stuff in this file wasn't here and instead relied on secondary/compiler-rt/BUILD.gn, and secondary/compiler-rt/lib/BUILD.gn would add a dep to scudo only if target_os == linux || fuchsia.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71131/new/
https://reviews.llvm.org/D71131
More information about the llvm-commits
mailing list