[PATCH] D27836: Store the "current position" index within the ASTRecordReader.
Richard Smith via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 16 17:14:42 PST 2016
rsmith added a comment.
I like moving `Idx` into the reader, but I have mixed feelings about the iterator-like interface. For consistency with the rest of the `ASTRecordReader` interface, and with how we model the writer side, I think perhaps `Record.ReadInt()` would fit better than using `*Record++`.
More information about the cfe-commits