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!

-- 
Arthur Cohen <arthur.cohen@embecosm.com>

Toolchain Engineer

Embecosm GmbH

Geschäftsführer: Jeremy Bennett
Niederlassung: Nürnberg
Handelsregister: HR-B 36368
www.embecosm.de

Fürther Str. 27
90429 Nürnberg


Tel.: 091 - 128 707 040
Fax: 091 - 128 707 077