[patch][rfc] Emit DeferredDeclsToEmit in order

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Jan 21 16:12:24 PST 2015


On 21 January 2015 at 16:25, Richard Smith <richard at metafoo.co.uk> 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?

Cheers,
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: t.patch
Type: text/x-patch
Size: 76369 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150121/705ae985/attachment.bin>


More information about the cfe-commits mailing list