Re: [PATCH 16/16] tcg: Remove TARGET_ALIGNED_ONLY

[Philippe Mathieu-Daudé]

On 2/5/23 18:08, Richard Henderson wrote:
> All uses have now been expunged.
>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   include/exec/memop.h  | 13 ++-----------
>   include/exec/poison.h |  1 -
>   tcg/tcg.c             |  5 -----
>   3 files changed, 2 insertions(+), 17 deletions(-)


> diff --git a/tcg/tcg.c b/tcg/tcg.c
> index cfd3262a4a..2ce9dba25c 100644
> --- a/tcg/tcg.c
> +++ b/tcg/tcg.c
> @@ -2071,13 +2071,8 @@ static const char * const ldst_name[] =
>   };
>   
>   static const char * const alignment_name[(MO_AMASK >> MO_ASHIFT) + 1] = {
> -#ifdef TARGET_ALIGNED_ONLY
> -    [MO_UNALN >> MO_ASHIFT]    = "un+",

Maybe we want to keep the "un+" prefix.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

> -    [MO_ALIGN >> MO_ASHIFT]    = "",
> -#else
>       [MO_UNALN >> MO_ASHIFT]    = "",
>       [MO_ALIGN >> MO_ASHIFT]    = "al+",
> -#endif
>       [MO_ALIGN_2 >> MO_ASHIFT]  = "al2+",
>       [MO_ALIGN_4 >> MO_ASHIFT]  = "al4+",
>       [MO_ALIGN_8 >> MO_ASHIFT]  = "al8+",