<div dir="ltr">LGTM</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 19, 2014 at 11:35 PM, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@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 dir="ltr">The attached patch updates CodeGen to add the llvm 'nonnull' attribute whenever the return or argument type is a reference (lvalue or rvalue reference).<div>
<br></div><div>There is more we could do here (for instance, 'this' pointers are also never null), but I'm starting here since the bulk of the change is merely updating the tests. In particular I added 'nonnull' whenever the mangled name indicated a reference type was involved. For Microsoft mangling tests and some Objective-C++ tests, I simply updated the tests with no attempt to reason about the correctness.</div>


<div><br></div><div>Please review!</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Nick</div><div><br></div></font></span></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>