<div class="gmail_quote">Sorry for the delay.</div><div class="gmail_quote"><br></div><div class="gmail_quote">On Fri, Jan 20, 2012 at 1:38 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">Hello, <div><br></div>The following patch adds __attribute__((no_address_safety_analysis)) which will allow to disable <div>
address safety analysis (such as e.g. AddressSanitizer or SAFECode) for a specific function.</div></blockquote><div><br></div><div>At a high level, I'm not thrilled with the name of the attribute. I'd really like to have a common prefix, or some other structured naming scheme. However, I see that this matches the pattern of the thread_safety attributes, which in turn match GCC attributes. Since this is the GCC attribute syntax, that makes sense.</div>
<div><br></div><div>For future reference, I'm hoping we can start enabling the use of C++11 attributes in the not-too-distant future with Clang, and at that point we'll want to have a namespace for any address safety (or related) attributes. Might be good to think about what those should look like.</div>
<div><br></div><div>The patch looks good with one exception: please add a template function and make sure that the instantiations pick up the attribute. I'm always a touch paranoid about the attribute/template interactions, although they're much better now than they were and everything should Just Work. ;] Feel free to commit if the template test cases work. =]</div>
</div>