mprobst marked 2 inline comments as done. mprobst added inline comments.
================ Comment at: clang/lib/Format/TokenAnnotator.cpp:3165 + } + } + ---------------- RKSimon wrote: > @mprobst - this is breaking buildbots: > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-android/builds/27196/steps/bootstrap%20clang/logs/stdio > > Please can you remove the default case and add a suitable llvm_unreachable() > after the switch statement ? Sorry for the breakage, and thanks to Jinsong Ji for fixing while I was OOO. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73335/new/ https://reviews.llvm.org/D73335 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
