I've read through the patch and things seem to make sense to me, but I'm afraid at this point you are the only one that fully understands all the corner cases, so I can't give more detailed comments.<div><br></div>

<div><div><br><div class="gmail_quote">On Wed, May 8, 2013 at 3:56 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Any thoughts on this?  This was a bunch of fun to work out.  =D<br>
<br>
The commit message I wrote is all wrong about the reinterpret_cast<br>
corner case.  For reinterpret_cast, you have to do null conversion,<br>
but all other conversions are impl-defined, and the easiest conversion<br>
is a no-op.<br>
<div><div class="h5"><br>
On Fri, May 3, 2013 at 6:59 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>     - Fix null handling for reinterpret casts.<br>
><br>
> Hi rjmccall,<br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D741" target="_blank">http://llvm-reviews.chandlerc.com/D741</a><br>
><br>
> CHANGE SINCE LAST DIFF<br>
>   <a href="http://llvm-reviews.chandlerc.com/D741?vs=1827&id=1828#toc" target="_blank">http://llvm-reviews.chandlerc.com/D741?vs=1827&id=1828#toc</a><br>
><br>
> Files:<br>
>   lib/CodeGen/MicrosoftCXXABI.cpp<br>
>   test/CodeGenCXX/microsoft-abi-member-pointers.cpp<br>
</div></div>_______________________________________________<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>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Joćo Matos
</div></div>