On Wed, Jan 21, 2015 at 4:12 PM, Rafael Espíndola <[email protected]> wrote: > On 21 January 2015 at 16:25, Richard Smith <[email protected]> wrote: >> You lose one nice property of the current model: if we emit 'f', and >> 'f' calls 'g', we then immediately emit 'g' as well. I think it'd be >> nicer to emit declarations in something like depth-first usage order. >> (Something like: swap DeferredDeclsToEmit with a local, then walk it >> emitting decls, and after each one, recursively call >> emitCurrentDeferredDecls if DeferredDeclsToEmit isn't empty.) > > I like it! > > What do you think of the attached patch?
Code change LGTM (typo "convinient") (I've not looked through all the test changes but the ones I looked through look like improvements.) _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
