[PATCH] D73734: [Fuchsia] Never link in implicit "system dependencies" of sanitizer runtimes
Petr Hosek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 30 12:10:28 PST 2020
phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.
LGTM % nit
================
Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:586
ArgStringList &CmdArgs) {
+ if (Triple.isOSFuchsia()) {
+ // Fuchsia never needs these. Any sanitizer runtimes with system
----------------
Nit: LLVM style says no braces for blocks with a single statement.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73734/new/
https://reviews.llvm.org/D73734
More information about the cfe-commits
mailing list