[LLVMdev] [PATCH] SelectionDAG Debugging
dag at cray.com
Fri Jan 15 14:34:07 PST 2010
On Friday 15 January 2010 16:23, Dan Gohman wrote:
> On Jan 15, 2010, at 1:38 PM, David Greene wrote:
> > On Friday 15 January 2010 15:09, Dan Gohman wrote:
> >>> Sound good? reimplement
> >> Unlimited-recursion dumping is what the existing dump routines
> >> already do, so it's a little odd to have a flag to allow these
> > Which existing dump routines are you referring to?
> dumpr(). I guess it wasn't commented. It is now :-).
Ah, one thing. dumpr uses DumpNodesr which does the "once" thing.
I actually would prefer a full dump. Perhaps we shouldn't try to
unify them, or maybe provide a flag to control behavior. In the
past separate APIs have been preferred over flags. I can rename
the ones I added to be more consistent with the existing stuff.
More information about the llvm-dev