[llvm-commits] [llvm] r93566 - /llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h

[Dan Gohman]

On Jan 15, 2010, at 2:38 PM, David Greene wrote:

> On Friday 15 January 2010 16:22, Dan Gohman wrote:
> 
>> +  /// dump - Dump this node, for debugging.
>>   void dump() const;
>> +  /// dump - Dump this node, for debugging.
>> +  /// The given SelectionDAG allows target-specific nodes to be printed
>> +  /// in human-readable form.
>>   void dump(const SelectionDAG *G) const;
> 
> Why not use default parameters?  Usually I don't like them but in this case
> it makes sense.  I can clean this up as I work through the other changes.

Default parameters are not debugger-friendly, with popular compilers
and debuggers.

Dan