[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