[PATCH] D15624: Add iOS/watchOS/tvOS support for ASan (clang part)

Alexey Samsonov via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 22 13:15:12 PST 2016


samsonov accepted this revision.
samsonov added a reviewer: samsonov.
samsonov added a comment.

LGTM


================
Comment at: lib/Driver/ToolChains.cpp:339
@@ +338,3 @@
+  }
+}
+
----------------
You might need `llvm_unreachable` at the very end to suppress GCC warnings (http://llvm.org/docs/CodingStandards.html#don-t-use-default-labels-in-fully-covered-switches-over-enumerations)


http://reviews.llvm.org/D15624





More information about the cfe-commits mailing list