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

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

[Markus Deuling]

Hello,

this is the next series of 6 patches to get rid of the macros in gdbarch.sh.

The following macros are replaced by this patches: 


* TARGET_VIRTUAL_FRAME_POINTER
* REGISTER_SIM_REGNO
* CONVERT_REGISTER_P, REGISTER_TO_VALUE and VALUE_TO_REGISTER
* SMASH_TEXT_ADDRESS
* *_REG_TO_REGNUM
* DEPRECATED_STACK_ALIGN


I tested this patchset by running gdb_mbuild.sh and run the testsuite on x86. The testsuite showed no regressions.
I'd appreciate any comments to this patches. Thank you very much in advance.

Regards,
Markus

-- 
Markus Deuling
GNU Toolchain for Linux on Cell BE
deuling@de.ibm.com

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

[Ulrich Weigand]

Markus Deuling wrote:

> The following macros are replaced by this patches: 
> 
> * TARGET_VIRTUAL_FRAME_POINTER
> * REGISTER_SIM_REGNO
> * CONVERT_REGISTER_P, REGISTER_TO_VALUE and VALUE_TO_REGISTER
> * SMASH_TEXT_ADDRESS
> * *_REG_TO_REGNUM
> * DEPRECATED_STACK_ALIGN

These look all good to me.  I've applied all six patches.

Thanks,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com

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

[Markus Deuling]

Ulrich Weigand wrote:
> Markus Deuling wrote:
> 
>> The following macros are replaced by this patches: 
>>
>> * TARGET_VIRTUAL_FRAME_POINTER
>> * REGISTER_SIM_REGNO
>> * CONVERT_REGISTER_P, REGISTER_TO_VALUE and VALUE_TO_REGISTER
>> * SMASH_TEXT_ADDRESS
>> * *_REG_TO_REGNUM
>> * DEPRECATED_STACK_ALIGN
> 
> These look all good to me.  I've applied all six patches.
> 
> Thanks,
> Ulrich
> 

Again, thank you very much for your effort :-)

-- 
Markus Deuling
GNU Toolchain for Linux on Cell BE
deuling@de.ibm.com

[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
deuling@de.ibm.com

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
  Ulrich.Weigand@de.ibm.com

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
deuling@de.ibm.com