[PATCH] D15631: Add dump() member function to ArgList.

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 17 17:35:38 PST 2015


echristo added a comment.

We appear to be wonderfully consistent about this. In general, I prefer dbgs for this sort of thing.

dzur:~/sources/llvm> grep -r errs\(\) * | wc -l
1461
dzur:~/sources/llvm> grep -r dbgs\(\) * | wc -l
4449

Seems to say dbgs in a very macro sort of way :)

Can also put a print method on Arg if you'd like as a small separate thing?

I'm not too picky, though if you don't do it I'll probably do it for consistency with everything else.

Thanks!

-eric


http://reviews.llvm.org/D15631





More information about the llvm-commits mailing list