Nice patch, LGTM.<div><br></div><div>I really like the clean up of all the pointer qualifier mangling into manglePointerQualifiers, makes the code much simpler to follow.</div><div><br></div><div><br></div><div>On Fri, Aug 31, 2012 at 12:55 PM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:</div>

<div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi John,<br>
<br>
Can you please review this patch?<br>
This fixes <a href="http://llvm.org/PR13444" target="_blank">http://llvm.org/PR13444</a> as well as a large part of<br>
<a href="http://llvm.org/PR13182" target="_blank">http://llvm.org/PR13182</a> plus adds quite a few tests.<br>
<br>
With this patch plus the pending PR13455 patch I don't get link-time<br>
problems when building SPEC CPU2006 tests anymore!<br>
(I'm only testing the files that are compileable with Clang++ though,<br>
e.g. no RTTI users etc)<br>
<br>
FTR, the SplitQualType / isa<ArrayType> part was mostly taken from<br>
ItaniumMangle.cpp<br>
<br>
Thanks in advance,<br>
Timur Iskhodzhanov,<br>
Google Russia<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>João Matos<br>
</div>