[U-Boot] [PATCH v2 1/5] kbuild: Enable -fshort-wchar

[Masahiro Yamada]

2017-09-09 19:47 GMT+09:00 Rob Clark <robdclark at gmail.com>:
> EFI_LOADER really wants UTF-16 strings (ie. %ls and L"string" are 16bit
> chars instead of 32bit chars).  But rather than enabling -fshort-wchar
> conditionally if EFI_LOADER is enabled, it was deemed preferrable to
> globally switch.
>
> Signed-off-by: Rob Clark <robdclark at gmail.com>
> ---
>  Makefile | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Makefile b/Makefile
> index 5bb3b09115..d635de12ab 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -360,6 +360,7 @@ KBUILD_CPPFLAGS := -D__KERNEL__ -D__UBOOT__
>  KBUILD_CFLAGS   := -Wall -Wstrict-prototypes \
>                    -Wno-format-security \
>                    -fno-builtin -ffreestanding
> +KBUILD_CFLAGS  += -fshort-wchar
>  KBUILD_AFLAGS   := -D__ASSEMBLY__
>
>  # Read UBOOTRELEASE from include/config/uboot.release (if it exists)
> --


Recently, Linux also switched to the global -fshort-wchar
and I think it makes more sense than giving it conditionally.


Acked-by: Masahiro Yamada <yamada.masahiro at socionext.com>



-- 
Best Regards
Masahiro Yamada