owenpan added a comment.

@kadircet thanks for reporting the crash and reverting the commit. I will fix 
it and reland the patch.



================
Comment at: clang/lib/Format/TokenAnnotator.cpp:1229
+          consumeToken();
+        assert(CurrentToken);
+        auto Previous = CurrentToken->getPreviousNonComment();
----------------
After `next()` or `consumeToken()`, `CurrentToken` would advance and could 
become null.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143755

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D143755: [clang-fo... Kadir Cetinkaya via Phabricator via cfe-commits
    • [PATCH] D143755: [cla... Owen Pan via Phabricator via cfe-commits

Reply via email to