Re: [PATCH v4 05/11] rules.mak: Add base-arch() rule

2020-06-05 Thread Philippe Mathieu-Daudé
On 6/1/20 9:26 AM, Philippe Mathieu-Daudé wrote: > On 6/1/20 4:13 AM, Richard Henderson wrote: >> On 5/22/20 9:37 AM, Philippe Mathieu-Daudé wrote: >>> + $(if $(call startwith,risc,$1),risc,\ >> >> Should be "riscv" not "risc". Especially the substitution. > > Indeed. > >> >>> +

Re: [PATCH v4 05/11] rules.mak: Add base-arch() rule

2020-06-01 Thread Philippe Mathieu-Daudé
On 6/1/20 4:13 AM, Richard Henderson wrote: > On 5/22/20 9:37 AM, Philippe Mathieu-Daudé wrote: >> + $(if $(call startwith,risc,$1),risc,\ > > Should be "riscv" not "risc". Especially the substitution. Indeed. > >> +$(if $(call startwith,aarch64,$1),arm,\ >

Re: [PATCH v4 05/11] rules.mak: Add base-arch() rule

2020-05-31 Thread Richard Henderson
On 5/22/20 9:37 AM, Philippe Mathieu-Daudé wrote: > + $(if $(call startwith,risc,$1),risc,\ Should be "riscv" not "risc". Especially the substitution. > + $(if $(call startwith,aarch64,$1),arm,\ > + $(if $(call startwith,x86_64,$1),i386

[PATCH v4 05/11] rules.mak: Add base-arch() rule

2020-05-22 Thread Philippe Mathieu-Daudé
Add a rule to return the base architecture for a QEMU target. The current list of TARGET_BASE_ARCH is: $ git grep TARGET_BASE_ARCH configure configure:7785:TARGET_BASE_ARCH="" configure:7795:TARGET_BASE_ARCH=i386 configure:7813:TARGET_BASE_ARCH=arm configure:7846:TARGET_BAS