[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/12] TCG/PPC: use TCG_REG_CALL_STACK instead o
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 08/12] TCG/PPC: use TCG_REG_CALL_STACK instead of TCG_REG_R1 |
Date: |
Sat, 2 Jul 2011 12:04:18 +0300 |
On Sat, Jul 2, 2011 at 12:06 AM, malc <address@hidden> wrote:
> On Sat, 2 Jul 2011, Blue Swirl wrote:
>
>> On Tue, Jun 28, 2011 at 1:51 AM, malc <address@hidden> wrote:
>> > On Sun, 26 Jun 2011, Blue Swirl wrote:
>> >
>> >> Use TCG_REG_CALL_STACK instead of TCG_REG_R1 etc. for consistency.
>> >>
>> >
>> > This i'd rather avoid.
>>
>> Why? In addition to the consistency among targets, a magic constant is
>> replaced with a symbol which improves the documenting abilities and
>> readability of the code.
>
> Makes it harder to read for me personally.
What about other people reading the code? With TCG_REG_CALL_STACK it's
pretty clear what is the purpose of the register, with '1' it is
assumed that the reader happens to know it. I don't think the exact
value is so interesting, or is register 1 somehow different from
others?