[rfc] [0/6] Remove macros from gdbarch.sh

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[rfc] [0/6] Remove macros from gdbarch.sh

Markus Deuling
Hello,

this set of patches are the first 5 patches to get rid of the macros in gdbarch.sh like TARGET_BYTE_ORDER etc.
These macros are replaced by their appropriate "gdbarch_" functions as it is already done with some macros in the past. I'd like to fulfill that transition and get completely rid of all macros.

The following macros are replaced by this patch set:

* CALL_DUMMY_LOCATION
* TARGET_BYTE_ORDER
* TARGET_CHAR_SIGNED
* TARGET_OSABI
* CANNOT_FETCH_REGISTER / CANNOT_STORE_REGISTER
* NUM_REGS / NUM_PSEUDO_REGS

I tested patches that change different tdep-files using the gdb_mbuild.sh script and run the testsuite
on x86. As far as I've seen it, there were no regressions. But I'm not sure if I caught up everything.

I'd appreciate any comments to this patches.  Do you think this would be OK to commit?
If so, then I'll come up with the next couple of patches.

Thank you very much in advance.

Regards,
Markus

--
Markus Deuling
GNU Toolchain for Linux on Cell BE
[hidden email]



Reply | Threaded
Open this post in threaded view
|

Re: [rfc] [0/6] Remove macros from gdbarch.sh

Ulrich Weigand
Markus Deuling wrote:

> The following macros are replaced by this patch set:
>
> * CALL_DUMMY_LOCATION
> * TARGET_BYTE_ORDER
> * TARGET_CHAR_SIGNED
> * TARGET_OSABI
> * CANNOT_FETCH_REGISTER / CANNOT_STORE_REGISTER
> * NUM_REGS / NUM_PSEUDO_REGS

I've committed (the updated versions of) all six patches now.

Thanks,
Ulrich

--
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  [hidden email]
Reply | Threaded
Open this post in threaded view
|

Re: [rfc] [0/6] Remove macros from gdbarch.sh

Markus Deuling
Ulrich Weigand wrote:

> Markus Deuling wrote:
>
>> The following macros are replaced by this patch set:
>>
>> * CALL_DUMMY_LOCATION
>> * TARGET_BYTE_ORDER
>> * TARGET_CHAR_SIGNED
>> * TARGET_OSABI
>> * CANNOT_FETCH_REGISTER / CANNOT_STORE_REGISTER
>> * NUM_REGS / NUM_PSEUDO_REGS
>
> I've committed (the updated versions of) all six patches now.
>
> Thanks,
> Ulrich
>

Thank you very much!


--
Markus Deuling
GNU Toolchain for Linux on Cell BE
[hidden email]