Alex, Does this mean that ASan is now supported on the iOS simulator?
Thanks, Anna. On Mar 15, 2013, at 9:13 AM, Alexander Potapenko <[email protected]> wrote: > +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
