>> +#import <Foundation/NSObject.h><div><br></div><div>Not sure if including <Foundation/NSObject.h> into a test is good idea. </div><div>Did you check that this tests works and passes on Linux? </div><div>
<br></div><div>--kcc <br><br><div class="gmail_quote">On Wed, Feb 1, 2012 at 7:27 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">
I've added a simple test into CodeGenObjCXX<br>
Looks like ObjC does not support the __attribute__ syntax, so it's<br>
unclear how to prevent a specific function from being instrumented.<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Jan 31, 2012 at 10:27 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
> On Tue, Jan 31, 2012 at 10:21 AM, Kostya Serebryany <<a href="mailto:kcc@google.com">kcc@google.com</a>> wrote:<br>
>><br>
>> Could you please also extend the test<br>
>> (cfe/trunk/test/CodeGen/address-safety-attr.cpp)?<br>
>> Ok for me, but I'd prefer to have 'ok' from someone else on the clang<br>
>> side.<br>
><br>
><br>
> Your comment is the same as mine -- we should add a test that would fail<br>
> without this change. Otherwise LGTM. =]<br>
<br>
<br>
<br>
</div></div><div class="HOEnZb"><div class="h5">--<br>
Alexander Potapenko<br>
Software Engineer<br>
Google Moscow<br>
</div></div></blockquote></div><br></div>