[llvm-commits] [llvm] r140156 - /llvm/trunk/lib/Transforms/Instrumentation/GCOVProfiling.cpp

[Nick Lewycky]

Devang Patel wrote:
> Author: dpatel
> Date: Tue Sep 20 13:05:45 2011
> New Revision: 140156
>
> URL: http://llvm.org/viewvc/llvm-project?rev=140156&view=rev
> Log:
> Update comment.
>
> Modified:
>      llvm/trunk/lib/Transforms/Instrumentation/GCOVProfiling.cpp
>
> Modified: llvm/trunk/lib/Transforms/Instrumentation/GCOVProfiling.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/GCOVProfiling.cpp?rev=140156&r1=140155&r2=140156&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/Instrumentation/GCOVProfiling.cpp (original)
> +++ llvm/trunk/lib/Transforms/Instrumentation/GCOVProfiling.cpp Tue Sep 20 13:05:45 2011
> @@ -158,8 +158,7 @@
>     class GCOVBlock;
>
>     // Constructed only by requesting it from a GCOVBlock, this object stores a
> -  // list of line numbers and a single filename, representing lines that belong
> -  // to the block.
> +  // list of line numbers representing lines that belong to the block.

How about "... list of line numbers within a single source file that 
belong to a block." ? As written, this sounds like a GCOVLines stores 
all the lines for a given GCOVBlock.

Nick

>     class GCOVLines : public GCOVRecord {
>      public:
>       void addLine(uint32_t Line) {
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>