Re: [tyndur-devel] [PATCH 10/11] cpu: Variablennamen die Register wiederspiegeln umbenannt

[Andreas Freimuth <m.nemo@xxxxxxx>]

On 13.01.2011 23:29, Kevin Wolf wrote:
> On Sun, Jan 09, 2011 at 08:33:32PM +0100, Andreas Freimuth wrote:
>> * Beispiel: rax und eax in ax umbenannt
>>
>> Signed-off-by: Andreas Freimuth <m.nemo@xxxxxxx>
> 
> Hm, meinst du wirklich, dass das eine gute Idee ist?
> 
> Ich glaube, wenn ich mal rumfragen würde, was sich die meisten unter ax
> vorstellen, wird die Mehrheit sich ein 16-Bit-Register vorstellen.
> Insofern verstehe ich den Sinn nicht ganz. Welchen Vorteil versprichst
> du dir denn von der Umbenennung?

Ziel war es das Duplizieren von Code zu vermeiden, bei dem sich nur die
Registernamen unterscheiden(z.B. Patch 9/11). Aber bluecode hat auch
schon im IRC angemerkt, dass Register im non-arch Teil nichts zu suchen
haben, wo mit er Recht hat. Von daher werde ich das ändern, und statt
dessen alles in den arch Teil verschieben.

> Übrigens kann ich diesen Patch nicht sauber anwenden, in amd64/cpu.c
> passt er angeblich nicht.

Hmm… das wundert mich. Aber aus irgend einem Grund sind es auch noch die
Patches mit den alten Commit-Messages und Makefile Änderungen.