[PATCH 3/3] Add a test case for the jit-reader interface.
Jan Kratochvil
jan.kratochvil@redhat.com
Wed Jan 9 12:37:00 GMT 2013
On Tue, 08 Jan 2013 21:40:02 +0100, Sanjoy Das wrote:
> > OK, that makes sense. But in such case please make there a clear separate
> > jit.h copy (with a different name and some added #ifdef/#error or something
> > like that protection so that if one mistakenly includes gdb/jit.h it gets
> > error-reported, with an explanation you made to me now above).
>
> Fixed.
> +#ifdef JIT_H
> +#error "We don't include jit.h directly since we'd like the jit-reader unit \
> + tests to break if we make ABI incompatible changes to the structures \
> + re-declared here."
> +#endif
OK, thanks for the explanation, it looks clear to me now.
> + gdb_test_no_output "jit-reader-load ${jit_reader_bin}"
Do not use full binary path in test name, those differ across testsuite runs.
That is use for example:
gdb_test_no_output "jit-reader-load ${jit_reader_bin}" "jit-reader-load"
Therefore these three parts are OK for check-in.
There was a never-replied mail but that suggestion is not required for this
check-in.
Re: [PATCH 1/3] Fix segfault when unwinding JIT frames using a custom reader.
http://sourceware.org/ml/gdb-patches/2012-12/msg00007.html
Message-ID: <20121201202522.GA22812@host2.jankratochvil.net>
Thanks,
Jan
More information about the Gdb-patches
mailing list