Re: [Qemu-devel] [PATCH] disas/microblaze: Remove unused REG_SP macro

[Philippe Mathieu-Daudé]

On 12/20/18 5:33 AM, Richard Henderson wrote:
> This causes a build error with debian sid, riscv64 host:
> 
> disas/microblaze.c:179: error: "REG_SP" redefined [-Werror]
>  #define REG_SP  1 /* stack pointer */
> 
> In file included from /usr/include/signal.h:306,
>                  from include/qemu/osdep.h:101,
>                  from disas/microblaze.c:36:
> /usr/include/riscv64-linux-gnu/sys/ucontext.h:36: note: this is the location 
> of the previous definition
>  # define REG_SP 2
> 
> Signed-off-by: Richard Henderson <address@hidden>
> ---
>  disas/microblaze.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/disas/microblaze.c b/disas/microblaze.c
> index 598ecbc89d..c23605043a 100644
> --- a/disas/microblaze.c
> +++ b/disas/microblaze.c
> @@ -176,7 +176,6 @@ enum microblaze_instr_type {
>  #define REG_TLBSX 36869 /* MMU: TLB Search Index reg */
>  
>  /* alternate names for gen purpose regs */
> -#define REG_SP  1 /* stack pointer */

This is clever than my previous suggestion:
https://lists.gnu.org/archive/html/qemu-devel/2018-08/msg00171.html

Reviewed-by: Philippe Mathieu-Daudé <address@hidden>

>  #define REG_ROSDP 2 /* read-only small data pointer */
>  #define REG_RWSDP 13 /* read-write small data pointer */
>  
>