Moved in r184321.

On Wed, Jun 19, 2013 at 12:45 PM, John McCall <[email protected]> wrote:

> On Jun 19, 2013, at 5:32 AM, Reid Kleckner <[email protected]> wrote:
>
> On Tue, Jun 18, 2013 at 8:07 PM, John McCall <[email protected]> wrote:
>
>> On Jun 9, 2013, at 9:45 AM, Reid Kleckner <[email protected]> wrote:
>> > Author: rnk
>> > Date: Sun Jun  9 11:45:02 2013
>> > New Revision: 183631
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=183631&view=rev
>> > Log:
>> > [CodeGen] Move EHScopeStack to CGCleanup.h from CodeGenFunction.h
>> >
>> > No functionality change.  CGCleanup.cpp provides the implementation for
>> > EHScopeStack, so it seems more consistent to place the class definition
>> > in CGCleanup.h.
>>
>> So, I specifically wasn't doing this because I didn't want to include
>> CGCleanup.h in every file in IR-gen, basically because I didn't want
>> random files depending on the cleanup-stack details.
>>
>
> Hm.  I needed this in order to reference EHScopeStack::stable_iterator
> from CallArgList in CGCall.h, which is included by CodeGenFunction.h.  I
> couldn't forward declare it, so this seemed cleaner.
>
> lib/CodeGen/CGEHScopeStack.h then?
>
>
> Just EHScopeStack.h, please.
>
> John.
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to