<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 30, 2013 at 10:45 AM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank" class="cremed">rnk@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm having a very hard time extracting this function from Sema and moving it over to AST/ItaniumCXXABI.cpp as John suggested.  It accesses a bunch of Sema fields.  Do you have any thoughts on how this should look in the long run?</blockquote>
</div><br>Just throwing this out there, but this kind of a refactoring and re-layering seems like a very reasonable thing to leave FIXME comments about (even file a PR if needed) and fix in follow-up commits rather than blocking other things on this change. Notably, not having any support for this causes us to not get very far testing other functionality.</div>
</div>