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

Ken Dyck Ken.Dyck at onsemi.com
Thu Nov 5 11:21:11 PST 2009


On Thursday, October 29, 2009 12:59 AM, Chris Lattner wrote:
> >> Please propose a patch that just uses the new methods you added to 
> >> simplify InitPreprocessor without changing the other 
> behavior.  That 
> >> will make it easier to review the patch and it will be 
> more obviously 
> >> correct (and can thus be applied without thinking) :)
> >
> > Is the attached patch what you have in mind? Or would you 
> like to see 
> > a more aggressive refactoring to remove all of the literal 
> type names 
> > and constant suffixes from InitPreprocessor.cpp?
> 
> This is a great start, applied as r85481.  Removing the rest 
> would be even better :)

Patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: InitPreprocessor-more-simplified.r86134.patch
Type: application/octet-stream
Size: 6375 bytes
Desc: InitPreprocessor-more-simplified.r86134.patch
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20091105/2d471ef2/attachment.obj>


More information about the cfe-dev mailing list