<div class="gmail_quote">On Tue, Nov 15, 2011 at 11:24 PM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Meanwhile, I removed -DADDRESS_SANITIZER from the patch. <div>Please take another look. <br><br>We can reinstate the macro or implement __has_feature/__has_extension when we decide what is the right thing to do.</div></blockquote>
<div><br></div><div>That seems like a fine plan to me.</div><div><br></div><div>One style nit on the patch:</div><div><br></div><div>Index: lib/Driver/Tools.cpp</div><div>===================================================================</div>
<div>--- lib/Driver/Tools.cpp<span class="Apple-tab-span" style="white-space:pre">      </span>(revision 144759)</div><div>+++ lib/Driver/Tools.cpp<span class="Apple-tab-span" style="white-space:pre">    </span>(working copy)</div>
<div>@@ -1701,6 +1701,11 @@</div><div>   if (getToolChain().SupportsProfiling())</div><div>     Args.AddLastArg(CmdArgs, options::OPT_pg);</div><div> </div><div>+  if (Args.hasFlag(options::OPT_faddress_sanitizer,</div><div>
+                   options::OPT_fno_address_sanitizer, false)) {</div><div>+    CmdArgs.push_back("-faddress-sanitizer");</div><div>+  }</div><div>+</div><div><br></div><div>No need for {}s here.</div><div><br>
</div><div>Otherwise, I think this looks good, but I would wait for John's OK before committing as he owns Clang's IRGen, and this seems like something he should sign off on. =] </div></div>