Re: [PATCH] D24751: [cleanup] Remove excessive padding from TextTokenRetokenizer::Position

2016-09-20 Thread Alexander Shaposhnikov via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL281995: [cleanup] Remove excessive padding from TextTokenRetokenizer::Position (authored by alexshap). Changed prior to commit: https://reviews.llvm.org/D24751?vs=71897=71963#toc Repository: rL LLVM

Re: [PATCH] D24751: [cleanup] Remove excessive padding from TextTokenRetokenizer::Position

2016-09-19 Thread David Majnemer via cfe-commits
majnemer added a subscriber: majnemer. majnemer accepted this revision. majnemer added a reviewer: majnemer. majnemer added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM https://reviews.llvm.org/D24751 ___

Re: [PATCH] D24751: [cleanup] Remove excessive padding from TextTokenRetokenizer::Position

2016-09-19 Thread Alexander Shaposhnikov via cfe-commits
alexshap added a comment. F2434937: Screen Shot 2016-09-19 at 7.33.38 PM.png the order BufferStart, BufferEnd,BufferPtr,BufferStartLoc,CurToken works as well Repository: rL LLVM https://reviews.llvm.org/D24751

[PATCH] D24751: [cleanup] Remove excessive padding from TextTokenRetokenizer::Position

2016-09-19 Thread Alexander Shaposhnikov via cfe-commits
alexshap created this revision. alexshap added reviewers: gribozavr, craig.topper. alexshap added a subscriber: cfe-commits. alexshap set the repository for this revision to rL LLVM. alexshap changed the visibility of this Differential Revision from "Public (No Login Required)" to "All Users".