<div class="gmail_quote">On Thu, Dec 1, 2011 at 1:28 AM, Alexander Potapenko <span dir="ltr"><<a href="mailto:glider@google.com">glider@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Attached the new version, PTAL<br>
<div class="im"><br>
> +  if (Args.hasArg(options::OPT_faddress_sanitizer)) {<br>
> Please<br>
> use Args.hasFlag(options::OPT_faddress_sanitizer, options::OPT_fno_address_sanitizer,<br>
> false))<br>
</div>Done<br>
<div class="im">><br>
> and hide the flag checking inside addAsanRTDarwinExe/etc<br>
</div>Then this should be something like "maybeAddAsan...", because this<br>
function is called unconditionally and other devs may think that we<br>
always add ASan.<br>
I've renamed the correspoding Linux function as well.<br></blockquote><div><br></div><div>I don't agree... The function is responsible for adding what Address Sanitizer needs, and it needs nothing if disabled. This matches several other 'addFoo' functions in the driver.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
> +const int AsanIfaceFnsSize = sizeof(AsanIfaceFns) / sizeof(char*);<br>
> +const char *AsanIfaceFns[13] = {<br>
> I would make these function-scope (if we end up needing them).<br>
</div>I've ended up using -undefined dynamic_lookup.<br>
Indeed, if someone has an undefined symbol in this dynamic library,<br>
let him catch this in another compiler mode.<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br>