On Wed, May 22, 2013 at 1:34 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, May 22, 2013 at 3:59 PM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br>
> +  // Pointer type qualifiers can only operate on pointer types, but not<br>
> +  // pointer-to-member types.<br>
> +  if (!Type->isPointerType() || Type->isMemberPointerType()) {<br>
><br>
> You don't need the isMemberPointerType here.<br>
<br>
</div>Removed.<br>
<div class="im"><br>
> This will still accept cases like:<br>
><br>
> typedef int *P;<br>
> P __ptr32 myp;<br>
><br>
> I would suggest checking isa<PointerType> on the type you get after<br>
> stripping off AttributedTypes.<br>
<br>
</div>I had an explicit test case in to ensure that worked because I felt it<br>
was a reasonable use case.  Are you saying we should not allow it?</blockquote><div><br></div><div>You said this was illegal in MSVC, so I don't see why we should allow it.</div></div>