+Ted, Anna On Fri, Mar 15, 2013 at 8:13 PM, Alexander Potapenko <[email protected]> wrote: > This patch lets the users invoke `clang -fsanitize=address` to link binaries > targeting the iOS simulator. > > http://llvm-reviews.chandlerc.com/D545 > > Files: > lib/Driver/ToolChains.cpp > > Index: lib/Driver/ToolChains.cpp > =================================================================== > --- lib/Driver/ToolChains.cpp > +++ lib/Driver/ToolChains.cpp > @@ -335,17 +335,19 @@ > // Add ASAN runtime library, if required. Dynamic libraries and bundles > // should not be linked with the runtime library. > if (Sanitize.needsAsanRt()) { > - if (Args.hasArg(options::OPT_dynamiclib) || > - Args.hasArg(options::OPT_bundle)) { > - // Assume the binary will provide the ASan runtime. > - } else if (isTargetIPhoneOS()) { > + if (isTargetIPhoneOS() && !isTargetIOSSimulator()) { > getDriver().Diag(diag::err_drv_clang_unsupported_per_platform) > << "-fsanitize=address"; > } else { > - AddLinkRuntimeLib(Args, CmdArgs, "libclang_rt.asan_osx_dynamic.dylib", > true); > - > - // The ASAN runtime library requires C++. > - AddCXXStdlibLibArgs(Args, CmdArgs); > + if (Args.hasArg(options::OPT_dynamiclib) || > + Args.hasArg(options::OPT_bundle)) { > + // Assume the binary will provide the ASan runtime. > + } else { > + AddLinkRuntimeLib(Args, CmdArgs, > + "libclang_rt.asan_osx_dynamic.dylib", true); > + // The ASAN runtime library requires C++. > + AddCXXStdlibLibArgs(Args, CmdArgs); > + } > } > }
-- Alexander Potapenko Software Engineer Google Moscow _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
