[cfe-dev] patch for generalizing stdint for n-bit bytes

[Ken Dyck]

On Monday, October 19, 2009 5:48 PM, Daniel wrote:
> On Mon, Oct 19, 2009 at 11:30 AM, Ken Dyck 
> <Ken.Dyck at onsemi.com> wrote:
> > On Saturday, October 17, 2009 8:17 PM, Ray wrote:
> >> This patch generalizes some of the defines produced by the 
> >> preprocessor in preparation for future patches to support of n-bit 
> >> bytes.
> >
> > Specifically, it adds support to stdint.h for all 
> mutiple-of-8 integer 
> > widths up to 64 bits, where it previously only supported 
> 8-, 16-, 32- 
> > and 64-bit widths.
> >
> >> Some other highlights include the addition of 40 new test 
> cases that 
> >> help better validate the preprocessor and target defines.  It also 
> >> gets rid of std::vector<char> in favor of 
> llvm::raw_ostream to create 
> >> defines.
> >
> > If it helps, I'd be happy to separate the new test cases and the 
> > raw_ostream conversion into separate patches.
> 
> Yes, that would probably be very helpful!

Attached are patches for the stdint.h/InitPreprocessor.cpp modifications and some new tests that exercise preprocessor initialization. 

I'll hold off on submitting a patch for the vector<char>-to-raw_ostream conversion in InitPreprocessor.cpp until the stdint.h patch is accepted/rejected, as there is too much overlap between the two to make separating them worthwhile. 

-Ken
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cpp-init-tests.r84633.patch
Type: application/octet-stream
Size: 97543 bytes
Desc: cpp-init-tests.r84633.patch
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20091020/21a4e79a/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: stdint-generalize.r84633.patch
Type: application/octet-stream
Size: 35780 bytes
Desc: stdint-generalize.r84633.patch
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20091020/21a4e79a/attachment-0001.obj>