Re: [RMC][PATCH v2] RMC: Support x32 build

[Todor Minchev <todor.minchev@linux.intel.com>]

On Wed, 2017-07-05 at 12:39 -0700, swee.aun.khor@intel.com wrote:
> From: sweeaun <swee.aun.khor@intel.com>
> 
> RMC efi is needed to compiled as 64 bits even DEFAULTTUNE is x32.
> 
> Signed-off-by: sweeaun <swee.aun.khor@intel.com>
> ---
>  Makefile.efi | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Makefile.efi b/Makefile.efi
> index 4f69dfe..95e0f8c 100644
> --- a/Makefile.efi
> +++ b/Makefile.efi
> @@ -16,6 +16,9 @@ RMC_INSTALL_LIB_PATH := $(RMC_INSTALL_PREFIX)/lib/
>  
>  RMC_CFLAGS := -DRMC_EFI -Wall -I$(TOPDIR)/inc -fpic -nostdinc -nostdlib  -fno-builtin -std=gnu90
>  
> +#RMC efi is needed to be compiled as 64 bit even DEFAULTTUNE is x32 
> +CC := $(patsubst -mx32, -m64, $(CC))
> +

Since this is not an RMC-specific problem, I believe that we shouldn't
patch the compiler flags in the RMC Makefile. This patch will provide
the desired outcome only when RMC is built with meta-intel. A better
alternative will be to split rmc.bb into rmc.bb (userspace app) and
rmc-efi.bb (efi library) and modify the compiler flags for each recipe.
I will send a patch that does this.

Todor

>  all: librmcefi
>  
>  $(RMC_LIB_OBJ): %.efi.o: %.c