<div class="gmail_extra"><div class="gmail_quote">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>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><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><br></div><div>I wonder if -relaxed-aliasing should trump -fthread-sanitizer, and fully suppress the TBAA... Not clear, but not a big deal to get addressed before you commit as it only impacts -fthread-sanitizer.</div>
</div></div>