<div dir="ltr">Committed as r222402. Thanks!<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 13, 2014 at 9:47 PM, Tomasz Miąsko <span dir="ltr"><<a href="mailto:tomasz.miasko@gmail.com" target="_blank">tomasz.miasko@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On czw, 2014-11-13 at 19:09 -0800, Richard Smith wrote:<br>
> On Thu, Nov 13, 2014 at 10:36 AM, Tomasz Miąsko<br>
> <<a href="mailto:tomasz.miasko@gmail.com">tomasz.miasko@gmail.com</a>> wrote:<br>
>         Indeed, something like "if (!member) return<br>
>         mangleExpression(base)", would be<br>
>         really nice, but the problem is that we need a member name<br>
>         from outermost<br>
>         MemberExpr generated from IndirectFieldDecl, and isArrow from<br>
>         innermost<br>
>         MemberExpr generated from IndirectFieldDecl.<br>
><br>
><br>
> Good point =)<br>
><br>
>         I updated the code to skip over all bases whose types are<br>
>         anonymous structs or<br>
>         unions, though I will mention that first version that relied<br>
>         on empty<br>
>         identifiers would also work.<br>
><br>
><br>
> Thanks, LGTM. Do you have an SVN account, or do you need someone to<br>
> commit this for you?<br>
><br>
<br>
</span>I don't have commit access, could you commit this for me?<br>
<br>
Thanks,<br>
Tomasz<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
>         <a href="http://reviews.llvm.org/D5997" target="_blank">http://reviews.llvm.org/D5997</a><br>
><br>
>         Files:<br>
>           lib/AST/ItaniumMangle.cpp<br>
>           test/CodeGenCXX/mangle-exprs.cpp<br>
><br>
><br>
><br>
<br>
<br>
</div></div></blockquote></div><br></div>