[LLVMdev] llvm-diff

[John McCall]

On Feb 28, 2011, at 3:36 AM, Renato Golin wrote:
> I'm considering adding debug metadata logic to llvm-diff, but it looks
> like some heavy refactoring will be needed to do so. Who's the owner
> of it?

Me.

> There is too much logic in DifferenceEngine, classes being declared in
> the cpp file, sub-classes of DifferenceEngine that could be used by
> the metadata diff, but with the current style
> (FunctionDifferenceEngine(*this)) will be hard to share.
> 
> I propose creating more files to split up the logic and use the
> appropriate h/cpp split, even if that takes more space. I'm willing to
> do some of this as part of my refactoring to get metadata diff, but I
> need to know from the owner if that's ok or if he/she has some other
> plans for the tool.

I have no objections to any of that.

John.