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

Masahiro Yamada
2017-09-11T01:57:09Z

2017-09-09 19:47 GMT+09:00 Rob Clark <robdclark@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@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@socionext.com>

--
Best Regards
Masahiro Yamada