Hi Wei-min,
On 9/26/2017 6:02 AM, Rainer Orth wrote:
Hi Pedro,
On 09/26/2017 10:33 AM, Rainer Orth wrote:
Apart from the capitalization differences that trip g++, the names
differ (PSR_VER vs. PSR_VERS), PSR_XCC is included in Solaris' PSR_RSV,
and there's no PSR_V8PLUS on Solaris either.
What you've done is fine with me to unbreak the build. Though
I'd prefer if we renamed those to avoid ever relying on host
symbols, anywhere. Like:
- #define PSR_S ...
+ #define SPARC64_PSR_S ...
etc.
agreed. That's probably best done when figuring out how to properly
separate target-dependent and independent parts of ADI support.
Rainer
Macros PSR_ICC/PSR_IMPL are not parts of ADI and were added to
sparc64-tdep.c
long before the Linux ADI project. Still trying to figure out why these two
macros,
pulled from /usr/include/v7/sys/psr.h, are causing problems now.
I suspect there were changes to gdb/defs.h, gdb/common/common-defs.h or
gnulib that ultimately dragged that file in post gdb-8.1. Seems I've
been barking up the wrong tree for this one, sorry.
Even so, the other problems with the ADI implemention mentioned both in
the patch submission and the PR are certainly new.
Rainer