cor3ntin added a comment.

I does think this patch probably makes sense but I agree with Erich that a 
comment is in order.
I'm also wondering whether we can end up in a situation where Tok and 
NextToken() get replaced by a single annotation in which case this would not 
work. I don't think so.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134334/new/

https://reviews.llvm.org/D134334

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to