ilya-biryukov added a comment. In D56723#1366885 <https://reviews.llvm.org/D56723#1366885>, @rsmith wrote:
> `ConsumeToken` is a fairly hot function; if you can avoid changes there > that'd be preferable. Done, there are no `enterUnknown` calls anymore and to avoid updating on each call to `ConsumeToken` we now store a location of the token we computed the expected type for. The resulting code looks ok to my taste, let me know what you think. Initially I thought this approach would allow to get rid of the RAII object to restore the type, but I don't see a good alternative to it even with the new approach. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56723/new/ https://reviews.llvm.org/D56723 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits