This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA:] Take 2 (was: Re: RFC: --enable-checking=valgrind.)


On Mon, Nov 18, 2002 at 09:24:19PM -0500, Hans-Peter Nilsson wrote:
> 	* Makefile.in (RUN_GEN, VALGRIND_DRIVER_DEFINES): New variables.
> 	(DRIVER_DEFINES): Add $(VALGRIND_DRIVER_DEFINES).
> 	(executing gencheck, genconfigs, genconditions, genflags,
> 	gencodes, genconstants, genemit, genrecog, genopinit, genextract,
> 	genpeep, genattr, genattrtab, genoutput, gengenrtl, genpreds,
> 	gengtype, genprotos): Prepend $(RUN_GEN).
> 	* configure.in: Move host compiler tests before --enable-checking
> 	tests.
> 	(--enable-checking=valgrind): New.
> 	* config.in, configure: Regenerate.
> 	* cppfiles.c (read_include_file) [ENABLE_VALGRIND_CHECKING]: When
> 	doing the mmap+1 trick, annotate the byte after the mmap:ed area
> 	as readable.
> 	(purge_cache) [ENABLE_VALGRIND_CHECKING]: Remove above annotation.
> 	* gcc.c (execute) [ENABLE_VALGRIND_CHECKING]: Arrange to prepend
> 	VALGRIND_PATH -q to each command.

Updated patch is ok.

Really, I don't think you needed to have combined the cpp fix with the
base valgrind invocation patch.


r~


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]