Bill, <div><br></div><div>Did you continue this discussion recently? </div><div>Any suggestions? </div><div><br></div><div>Thanks, </div><div><br></div><div>--kcc <br><br><div class="gmail_quote">On Thu, Aug 23, 2012 at 4:50 PM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">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">"Encoding Compile Flags into the IR"? <div>Yes, that's the same problem. </div><div>Great! I can wait. </div>
<div><br></div><div>--kcc </div><div class="HOEnZb"><div class="h5"><div><br><div class="gmail_quote">On Wed, Aug 22, 2012 at 9:55 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@apple.com" target="_blank">echristo@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><br>
On Aug 22, 2012, at 12:20 AM, Kostya Serebryany <<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>> wrote:<br>
<br>
> Hello,<br>
><br>
> I'd like to resurrect an old discussion: how to selectively disable ThreadSanitizer (or AddressSanitizer, etc) instrumentation<br>
> for particular functions.<br>
> For AddressSanitizer we currently use a function attribute "AddressSafety" (if it is not set, instrumentation does not happen),<br>
> but Chris doesn't like this idea (attributes are a scarce resource) and he suggested to use module-level metadata instead of an attribute.<br>
><br>
> The attached clang patch adds such metadata node for all functions when -fthread-sanitizer is given.<br>
> If that is what we want, I'll gradually change AddressSanitizer to use similar approach and will kill the "AddressSafety" attribute.<br>
><br>
> BTW, the code looks like this:<br>
>   llvm::Value *Fn[] = { F }; ...<br>
>   llvm::MDNode::get(VMContext, Fn);<br>
> Do we want this code to look like<br>
>   llvm::MDNode::get(VMContext, F);<br>
> (i.e. to add more llvm::MDNode::get() methods)?<br>
<br>
</div></div>Bill has been working on something similar for command line options and LTO. There's still been some chatting about it, but Bill's planning on updating the list in the next few days with what's going on if waiting a bit is ok?<br>


<span><font color="#888888"><br>
-eric<br>
<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>