This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] gdb passes and returns incorrect values when dealing with small array on Sparc
On 03/29/2017 02:56 AM, Pedro Alves wrote:
Hi Vladimir,
gdb development happens on a separate list - gdb-patches@. Please resend
the patch there. TIA!
Done.
A question I'll have is whether this had any observable effect in testsuite
results, other than "no regressions". I.e., whether there were progressions,
or whether we're missing testsuite coverage.
6 tests ( from gdb/testsuite/gdb.base/gnu_vector.exp) failed on
sparc64-Linux and on sparc-Solaris in 32- and 64-bit mode.
Now all these tests passed.
gdb/testsuite/gdb.base/gnu_vector.exp has 117 different cases for small
(and not small) arrays and structures.
I think we don't need an additional test in gdb/testsuite.
-Vladimir
On 03/28/2017 06:21 PM, vladimir.mezentsev@oracle.com wrote:
From: Vladimir Mezentsev <vladimir.mezentsev@oracle.com>
gdb has a special type (TYPE_CODE_ARRAY) to support the gcc extension
(https://gcc.gnu.org/onlinedocs/gcc/Vector-Extensions.html).
TYPE_CODE_ARRAY is handled incorrectly for both (32- and 64-bit) modes on Sparc machines.
Tested on sparc64-linux-gnu and sparc-solaris (32- and 64-bit mode).
No regressions.
2017-03-27 Vladimir Mezentsev <vladimir.mezentsev@oracle.com>
* gdb/sparc-tdep.c (sparc_structure_return_p): New function.
Also, please drop "gdb/" here. Filename entries in ChangeLogs
are relative to the ChangeLog file path.
Thanks,
Pedro Alves