[PATCH] implement -fno-short-wchar option

Rafael EspĂ­ndola rafael.espindola at gmail.com
Mon Feb 24 10:03:38 PST 2014


LGTM

On 24 February 2014 10:55, Richard Barton <richard.barton at arm.com> wrote:
> Hello reviewers
>
> The attached patch implements the -fno-short-wchar option for clang.
>
> I have followed the model for -fshort-wchar by only adding it to the cc1
> line if it is actually asked for, and leaving the default to sort itself
> out.
>
> Testing added to show that it generates the right compiler defines when put
> down. This only checks __SIZEOF_WCHAR_T and __WCHAR_T_WIDTH as the other two
> are different depending on whether the default is signed or unsigned. There
> is also a new test to make sure they turn each other off.
>
> Please review
>
> Thanks
> Rich
>
> Richard Barton
> Staff Engineer - LLVM
> ARM Ltd, Cambridge
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



More information about the cfe-commits mailing list