This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [pushed] s390-vregs.exp: Fix Tcl error after non-zero-pad patch
- From: Andreas Arnez <arnez at linux dot vnet dot ibm dot com>
- To: Tom Tromey <tom at tromey dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 01 Aug 2017 19:57:12 +0200
- Subject: Re: [pushed] s390-vregs.exp: Fix Tcl error after non-zero-pad patch
- Authentication-results: sourceware.org; auth=none
- References: <m34lu13i48.fsf@oc1027705133.ibm.com> <87wp6o8dh8.fsf@tromey.com>
On Mon, Jul 31 2017, Tom Tromey wrote:
>>>>>> "Andreas" == Andreas Arnez <arnez@linux.vnet.ibm.com> writes:
>
> Andreas> This is a regression, caused by commit 30a254669b16b8 -- "Don't always
> Andreas> zero pad in print_*_chars". That patch introduced a new procedure
> Andreas> "hex128" for formatting a 128-bit value as hex, but it accidentally moved
> Andreas> the calculation of the 128-bit value into that new procedure as well
> Andreas> instead of leaving it in the original context. This is fixed.
>
> Thanks. I'm sorry about that. I try not to make mistakes like this by
> running all my changes through the buildbot, but either it didn't report
> this, or I misread the results.
Sure, no problem.
Note that this test only runs on systems with a vector facility. And
even then your patch didn't cause additional FAILs, but a Tcl error
instead.
--
Andreas