<p dir="ltr"><br>
On Jun 18, 2013 5:34 PM, "John McCall" <<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>> wrote:<br>
><br>
> On Jun 18, 2013, at 5:11 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
> > On Tue, Jun 18, 2013 at 5:07 PM, John McCall <<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>> wrote:<br>
> >> On Jun 9, 2013, at 9:45 AM, Reid Kleckner <<a href="mailto:reid@kleckner.net">reid@kleckner.net</a>> wrote:<br>
> >>> Author: rnk<br>
> >>> Date: Sun Jun  9 11:45:02 2013<br>
> >>> New Revision: 183631<br>
> >>><br>
> >>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=183631&view=rev">http://llvm.org/viewvc/llvm-project?rev=183631&view=rev</a><br>
> >>> Log:<br>
> >>> [CodeGen] Move EHScopeStack to CGCleanup.h from CodeGenFunction.h<br>
> >>><br>
> >>> No functionality change.  CGCleanup.cpp provides the implementation for<br>
> >>> EHScopeStack, so it seems more consistent to place the class definition<br>
> >>> in CGCleanup.h.<br>
> >><br>
> >> So, I specifically wasn't doing this because I didn't want to include<br>
> >> CGCleanup.h in every file in IR-gen, basically because I didn't want<br>
> >> random files depending on the cleanup-stack details.<br>
> ><br>
> > So, should we just pull out EHScopeStack into it's own header &<br>
> > implementation file?<br>
><br>
> I don't know why specifically you care about giving it its own<br>
> implementation file, but putting it in its own header isn't unreasonable.</p>
<p dir="ltr">Don't have much of as dog in this race,but I tend to find code easier to navigate when its a one to one mapping.</p>
<p dir="ltr">><br>
> John.<br>
</p>