Re: [PATCH, RFC] Dumping expanded MD files

2012-12-04 Thread Kirill Yukhin
Hello > Ok. That was checked in: http://gcc.gnu.org/ml/gcc-cvs/2012-12/msg00075.html http://gcc.gnu.org/ml/gcc-cvs/2012-12/msg00077.html Thanks, K

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-30 Thread Richard Henderson
On 2012-11-30 01:03, Michael Zolotukhin wrote: > Changelog: > 2012-11-30 Michael Zolotukhin > > * Makefile.in: Add target mddump, build/genmddump.o. Extend > genprogrtl with mddump. > * genmddump.c: New. Ok. r~

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-30 Thread Michael Zolotukhin
I didn't find any dump_rtx - instead, I used print_inline_rtx which seems to be what we need to output to stdout, instead of stderr. Updated version of the patch is attached. Ok for trunk? Changelog: 2012-11-30 Michael Zolotukhin * Makefile.in: Add target mddump, build/genmddump.o. Ex

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-29 Thread Richard Henderson
On 2012-11-28 23:22, Michael Zolotukhin wrote: > + debug_rtx (desc); ... oh, I see. You wanted dump_rtx here, so you can output to stdout. r~

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-29 Thread Richard Henderson
On 2012-11-28 23:22, Michael Zolotukhin wrote: > +.PHONY: mddump > +mddump: $(BUILD_RTL) $(MD_DEPS) build/genmddump$(build_exeext) > + $(RUN_GEN) build/genmddump$(build_exeext) $(md_file) 2> tmp-mddump.md Why in the world are you dumping to stderr instead of stdout? r~

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-28 Thread Michael Zolotukhin
Thanks for the input, the patch seems to be much more cute now. Do we still need to play with release/non-release builds, or is it ok to commit this patch to the trunk as it is? Changelog: 2012-11-29 Michael Zolotukhin * Makefile.in: Add target mddump, build/genmddump.o. Extend

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-27 Thread Richard Henderson
On 11/22/2012 09:48 AM, Kirill Yukhin wrote: > +.PHONY: s-mddump > +s-mddump: $(BUILD_RTL) $(MD_DEPS) build/genmddump$(build_exeext) > + $(RUN_GEN) build/genmddump$(build_exeext) $(md_file) 2> tmp-mddump.md I think just mddump: ... $(RUN_GEN) ... > mddump will be sufficient. This i

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-23 Thread Hans-Peter Nilsson
On Thu, 22 Nov 2012, Kirill Yukhin wrote: > Hi folks, > > This patch adds an utility for dumping MD-files after iterators and > define_substs expanding. The new utility is named genmddump and is > built along with other gen* programs. > > I also added new target to Makefile to invoke this utility.

Re: [PATCH, RFC] Dumping expanded MD files

2012-11-23 Thread Mike Stump
On Nov 22, 2012, at 9:48 AM, Kirill Yukhin wrote: > This patch adds an utility for dumping MD-files after iterators and > define_substs expanding. > What do you guys think? Cool. (That's not an official review. :-))

[PATCH, RFC] Dumping expanded MD files

2012-11-22 Thread Kirill Yukhin
Hi folks, This patch adds an utility for dumping MD-files after iterators and define_substs expanding. The new utility is named genmddump and is built along with other gen* programs. I also added new target to Makefile to invoke this utility. However, I’m not sure it’s general enough to be added