Messages in this thread | | | Date | Mon, 19 Oct 2020 15:24:41 -0500 | From | Segher Boessenkool <> | Subject | Re: [PATCH 3/3] powerpc: Fix pre-update addressing in inline assembly |
| |
On Mon, Oct 19, 2020 at 12:12:48PM +0000, Christophe Leroy wrote: > In several places, inline assembly uses the "%Un" modifier > to enable the use of instruction with pre-update addressing,
Calling this "pre-update" is misleading: the register is not updated before the address is generated (or the memory access done!), and the addressing is exactly the same as the "non-u" insn would use. It is called an "update form" instruction, because (at the same time as doing the memory access, logically anyway) it writes back the address used to the base register.
> but the associated "<>" constraint is missing.
But that is just fine. Pointless, sure, but not a bug.
> Use UPD_CONSTR macro everywhere %Un modifier is used.
Eww. My poor stomach.
Have you verified that update form is *correct* in all these, and that we even *want* this there?
Segher
| |