| Message ID | 20221216170118.457649-1-dmalcolm@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | gccrs: add selftest-rust-gdb and selftest-rust-valgrind "make" targets | expand |
Hi David, On 12/16/22 18:01, David Malcolm wrote: > Add "make" targets to make it easy to run the rust selftests under gdb > and under valgrind via: > make selftest-rust-gdb > and > make selftest-rust-valgrind > respectively, similar to analogous "make" targets in the C and C++ > frontends. > > Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu. > > OK for trunk? > > gcc/rust/ChangeLog: > * Make-lang.in (selftest-rust-gdb): New. > (selftest-rust-valgrind): New. > > Signed-off-by: David Malcolm <dmalcolm@redhat.com> > --- > gcc/rust/Make-lang.in | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/gcc/rust/Make-lang.in b/gcc/rust/Make-lang.in > index 681ac7b3fee..76015b3426b 100644 > --- a/gcc/rust/Make-lang.in > +++ b/gcc/rust/Make-lang.in > @@ -275,6 +275,18 @@ s-selftest-rust: $(RUST_SELFTEST_DEPS) > $(GCC_FOR_TARGET) $(RUST_SELFTEST_FLAGS) > $(STAMP) $@ > > +# Convenience methods for running rust selftests under gdb: > +.PHONY: selftest-rust-gdb > +selftest-rust-gdb: $(RUST_SELFTEST_DEPS) > + $(GCC_FOR_TARGET) $(RUST_SELFTEST_FLAGS) \ > + -wrapper gdb,--args > + > +# Convenience methods for running rust selftests under valgrind: > +.PHONY: selftest-rust-valgrind > +selftest-rust-valgrind: $(RUST_SELFTEST_DEPS) > + $(GCC_FOR_TARGET) $(RUST_SELFTEST_FLAGS) \ > + -wrapper valgrind,--leak-check=full > + > # Install info documentation for the front end, if it is present in the source directory. This target > # should have dependencies on info files that should be installed. > rust.install-info: OK for trunk :) Thanks again!
diff --git a/gcc/rust/Make-lang.in b/gcc/rust/Make-lang.in index 681ac7b3fee..76015b3426b 100644 --- a/gcc/rust/Make-lang.in +++ b/gcc/rust/Make-lang.in @@ -275,6 +275,18 @@ s-selftest-rust: $(RUST_SELFTEST_DEPS) $(GCC_FOR_TARGET) $(RUST_SELFTEST_FLAGS) $(STAMP) $@ +# Convenience methods for running rust selftests under gdb: +.PHONY: selftest-rust-gdb +selftest-rust-gdb: $(RUST_SELFTEST_DEPS) + $(GCC_FOR_TARGET) $(RUST_SELFTEST_FLAGS) \ + -wrapper gdb,--args + +# Convenience methods for running rust selftests under valgrind: +.PHONY: selftest-rust-valgrind +selftest-rust-valgrind: $(RUST_SELFTEST_DEPS) + $(GCC_FOR_TARGET) $(RUST_SELFTEST_FLAGS) \ + -wrapper valgrind,--leak-check=full + # Install info documentation for the front end, if it is present in the source directory. This target # should have dependencies on info files that should be installed. rust.install-info:
Add "make" targets to make it easy to run the rust selftests under gdb and under valgrind via: make selftest-rust-gdb and make selftest-rust-valgrind respectively, similar to analogous "make" targets in the C and C++ frontends. Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu. OK for trunk? gcc/rust/ChangeLog: * Make-lang.in (selftest-rust-gdb): New. (selftest-rust-valgrind): New. Signed-off-by: David Malcolm <dmalcolm@redhat.com> --- gcc/rust/Make-lang.in | 12 ++++++++++++ 1 file changed, 12 insertions(+)