[llvm-commits] [llvm] r109739 - in /llvm/trunk/tools/llvm-diff: ./ CMakeLists.txt DifferenceEngine.cpp DifferenceEngine.h Makefile llvm-diff.cpp

[Chris Lattner]

On Jul 29, 2010, at 12:53 AM, John McCall wrote:

> Author: rjmccall
> Date: Thu Jul 29 02:53:27 2010
> New Revision: 109739
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=109739&view=rev
> Log:
> Add the llvm-diff tool, which performs a relatively naive structural
> diff of a function.  There's a lot of cruft in the current version, and
> it's pretty far from perfect, but it's usable.
> 
> Currently only capable of comparing functions.  Currently ignores metadata.
> Currently ignores most attributes of functions and instructions.
> 
> Patches welcome.

Very cool John.

Does this actually work?  What are its advantages over diff?  Please add a couple of simple tests to llvm/test, and a man page in llvm/docs/CommandGuide/.  A demo of how it works would be nice too :).

It doesn't matter much to me, but the large methods in DiffConsumer should probably be moved out of line so that it is easier to understand the class.

-Chris