<div class="gmail_quote">On Mon, Dec 13, 2010 at 2:27 PM, Abramo Bagnara <span dir="ltr"><<a href="mailto:abramo.bagnara@gmail.com">abramo.bagnara@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Skip ParenType on function instantiations.</blockquote></div><br><div>While this fixes one case, your patch remains fundamentally broken. There are more cases throughout Sema that reason about a QualType or TypeSourceInfo or TypeLoc and expect getAs<FunctionType> or dyn_cast<FunctionType> to Just Work. They don't any more. Function attributes won't work now for example. Can you audit the codebase looking for these? They should be very easy to craft test cases for.</div>
<div><br></div><div>Also, I would appreciate at least the initial check for whether the type (or type loc) is a ParenType to be in the inline section of code. The overwhelming common case is that this is a no-op, and we don't want to force a function call just to find that out.</div>