[llvm-commits] [llvm] r112812 - in /llvm/trunk: lib/VMCore/AsmWriter.cpp test/Transforms/IndVarSimplify/2003-12-10-IndVarDeadCode.ll

[Chris Lattner]

On Sep 2, 2010, at 9:43 AM, Dan Gohman wrote:

> 
> On Sep 2, 2010, at 1:52 AM, Duncan Sands wrote:
> 
>> Author: baldrick
>> Date: Thu Sep  2 03:52:23 2010
>> New Revision: 112812
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=112812&view=rev
>> Log:
>> Print the number of uses of a function in the .ll since it can be informative
>> and there seems to be no reason not to.
> 
> Hi Duncan,
> 
> IMHO, the AsmPrinter should just print the IR. It shouldn't be doing
> extra unnecessary analysis (especially not when the analysis has
> quadratic worst-case complexity :-}). There has been a fair amount of
> confusion from newcomers about whether these numbers are an explicit
> part of the IR; they aren't.

I agree, to me, this is just clutter.

> Another possibility is to suppress use counts for single-use
> instructions 

This sounds like a great idea!

-Chris