|
From: | Richard Henderson |
Subject: | Re: [PATCH 1/2] target/arm: Fix handling of LDAPR/STLR with negative offset |
Date: | Tue, 9 Jul 2024 08:45:27 -0700 |
User-agent: | Mozilla Thunderbird |
On 7/9/24 07:11, Philippe Mathieu-Daudé wrote:
BTW I noted some instr formats use 'uimm*' for unsigned immediate. Maybe we could recommend/enforce that, having 'imm*' always signed.
With Arm, especially with load/store, some insns have unsigned fields and some have signed fields, but for implementation purposes we need to use the same argument set. Therefore we cannot mix names like this.
This naming convention works better with more regular encodings such as riscv or loongarch. r~
[Prev in Thread] | Current Thread | [Next in Thread] |