[PATCH v2] locale: update _nl_C_LC_CTYPE_class and _nl_C_LC_CTYPE_class32 arrays to have types uint16_t and uint32_t respectively

Adhemerval Zanella adhemerval.zanella@linaro.org
Thu Apr 1 18:55:15 GMT 2021



On 01/04/2021 14:29, Lirong Yuan via Libc-alpha wrote:
> Thanks for the information! I have applied for a GCC Compile Farm account.
> 
> "Since account moderation is done manually, it may take a few days before
> the request is reviewed."
> 
> Regards,
> Lirong

I have checked this patch on powerpc64-linux-gnu and unfortunately it breaks
locale.  Running localedef:

$ ./testrun.sh /home/azanella/projects/glibc/build/powerpc64-linux-gnu/locale/localedef --quiet -c -f UTF-8 -i en_US         /home/azanella/projects/g
/home/azanella/projects/glibc/build/powerpc64-linux-gnu/locale/localedef: invalid option -- 'c'
Try `localedef --help' or `localedef --usage' for more information.

It fails on the argp_parse routine (I haven't dig into to check where exactly).

The first version that uses _alias does not trigger any regression so I think
it would be a better option.  Another option is to define the arrays with a
macro that define it in the same layout regardless of the endianess, but the
alignas macro is clearly a simpler approach.



More information about the Libc-alpha mailing list