<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt">r169201<br><br><div class="gmail_quote">On Sat, Dec 1, 2012 at 9:46 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"><br>
  LGTM<br>
<div class="im"><br>
<br>
================<br>
Comment at: lib/asan/lit_tests/use-after-scope-inlined.cc:1<br>
@@ +1,2 @@<br>
+// Test with "-O2" only to make sure inlining (leading to use-after-scope)<br>
+// happens. "always_inline" is not enough, as Clang doesn't emit<br>
----------------<br>
</div><div class="im">Alexey Samsonov wrote:<br>
> Kostya Serebryany wrote:<br>
> > use-after-scope feature deserves a more extensive set test,<br>
> > probably in form of large list of DEATH_TESTS (like in asan_test.cc)<br>
> > in a separate file, asan_lifetime_test.cc<br>
> > That could go as a separate commit.<br>
> I agree that I should better start writing these tests :) Anyway, they should go in when we teach Clang to emit lifetime markers to catch actual use-after-scope bugs.<br>
</div>Cases with inlining seem to work fine today. Why wait?<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D141" target="_blank">http://llvm-reviews.chandlerc.com/D141</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>
</div>