Re: [PATCH] [powerpc] Tighten contraints for asm constant parameters

[Florian Weimer <fweimer@redhat.com>]

* Paul A. Clarke:

> I was too loose with my characterization of 'i', in contrast to 'n':
>
> | ‘n’
> |      An immediate integer operand with a known numeric value is allowed.
> |      Many systems cannot support assembly-time constants for operands less
> |      than a word wide. Constraints for these operands should use ‘n’
> |      rather than ‘i’.
>
> The cases changed by the patch require a *known numeric value*, as they are
> used as immediate values (hardcoded in the generated opcode).
>
> I will reword to:
>   There are a few places where only known numeric values are acceptable for
>   `asm` parameters, yet the constraint "i" is used.  "i" can include
>   "symbolic constants whose values will be known only at assembly time or
>   later."
>
>   Use "n" instead of "i" where known numeric values are required.
>
> Does that work better?

Yes, that explains the difference.  Thanks!

Florian