<div class="gmail_extra">and committed: r155430.<br><br><div class="gmail_quote">On Tue, Apr 24, 2012 at 10:57 AM, 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"><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Tue, Apr 24, 2012 at 9:55 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra"><div class="gmail_quote"><div>On Mon, Apr 23, 2012 at 1:09 AM, 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">
Hello, <div>Please review the following updated patch. </div><div>TBAA should be generated for ThreadSanitizer regardless of opt level and relaxed-aliasing. </div></blockquote><div><br></div></div><div>Since Chris has goon a bit silent, I'll pick this up. I think you've addressed his feedback already.</div>


<div><br></div><div>Couple of tiny nits, and with these fixes, please commit.</div><div><br></div><div><div>+// when -fthread-sanitizer is used TBAA should be generated at all opt levels</div><div>+// even if -relaxed-aliasing is present.</div>


</div><div><br></div><div>s/when/When/</div></div></div></blockquote></div><div>Done.  </div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra">
<div class="gmail_quote">
<div><br></div><div>Can you add a negative test that demonstrates we *don't* generate this TBAA in -O0 or -O2 -relaxed-aliasing?</div></div></div></blockquote></div><div>Done.  </div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>I wonder if -relaxed-aliasing should trump -fthread-sanitizer, and fully suppress the TBAA... </div></div></div></blockquote><div><br></div></div><div>
Then those using -relaxed-aliasing and -fthread-sanitizer will be getting undesirable warnings. </div>
<div>BTW, we *are* using -relaxed-aliasing :) </div><div><br></div><div>Thanks! </div><div><br></div><div>--kcc </div><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="gmail_extra"><div class="gmail_quote"><div>Not clear, but not a big deal to get addressed before you commit as it only impacts -fthread-sanitizer.</div>
</div></div>
</blockquote></div></div><br></div>
</blockquote></div><br></div>