LGTM
On 24 February 2014 10:55, Richard Barton <[email protected]> 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 > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
