<div dir="ltr">Hello John!<div><br><div>Finally moved this to SemaType and added more tests for pointers to function appearing within other declarations.</div><div>Could you, please, look at this?<br><div class="gmail_extra">

<br><div class="gmail_quote">On 10 April 2013 22:40, John McCall <span dir="ltr"><<a href="mailto:rjmccall@apple.com" target="_blank">rjmccall@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Apr 8, 2013, at 3:06 PM, Alexander Zinenko <<a href="mailto:ftynse@gmail.com">ftynse@gmail.com</a>> wrote:<br>
> Sure, this code should belong to SemaDeclAttr. It required several not very elegant changes to make it work correctly in some places.<br>
> I also removed 'if'-ed dyn_cast to FieldDecl used just to get type since it is inherited from ValueDecl and should be covered in the previous 'if'.<br>
<br>
</div>Mmm.  We generally handle this as a type attribute, not a decl attribute, so I think you probably need to handle it in SemaType.  This will let you handle things like variables declared as pointers to __thiscall functions.  It should just be a matter of looking at the immediate declarator context when applying the attribute (and inelegantly repeating some small amount of analysis about CurContext and 'static' and so forth).<br>


<span class="HOEnZb"><font color="#888888"><br>
John.<br>
</font></span></blockquote></div><br></div></div></div></div>