[cfe-commits] [PATCH] Refactoring of vtable and VTT builders

[Peter Collingbourne]

On Mon, Jun 13, 2011 at 03:19:11PM -0700, Anders Carlsson wrote:
> 
> On Jun 13, 2011, at 12:41 PM, Peter Collingbourne wrote:
> 
> > Hi,
> > 
> > This patch refactors the vtable and VTT builders to provide a
> > backend-neutral interface to the builders in the AST library, which
> > CodeGen uses to build vtables and VTTs in LLVM.  No functionality
> > change intended.
> 
> While I'm not crazy about this idea, other people (Doug) are in favor of it, so I'm reluctantly fine with it :)
> 
> That being said, I don't think this patch should be committed in its current state. This should be done in incremental steps,
> there's no reason to move both the VTT builder and VTable builder in a single commit.

OK, then.  Following your and Eli's suggestions I will try to
break this patch up into the following steps:

1) Move GlobalDecl to AST
2) Make VTTBuilder independent of CodeGen (broken into sub-steps)
3) Move VTTBuilder to AST
4) Make VTableBuilder independent of CodeGen (broken into sub-steps)
5) Move VTableBuilder to AST

As a start, I am attaching a (simple) patch for 1.  OK to commit?

> Also, the files should be svn cp'ed to preserve history; being able to easily go back and see the rationale behind changes is
> crucial to fixing bugs/regressions in this code.

Hmm, I'll see if I can figure out how to persuade git-svn to do that.

Thanks,
-- 
Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Move-GlobalDecl-to-AST.patch
Type: text/x-diff
Size: 4841 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110614/a9d15040/attachment.patch>