aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
Please fix the clang format issue as well. Other than that and a nit, this LGTM. ================ Comment at: clang/lib/Lex/PPMacroExpansion.cpp:1587 + // The `minus` operator is the next token we read from the stream. + auto Toks = std::make_unique<Token[]>(2); + OS << "-"; ---------------- Pretty sure `2` can be `1` so that we only create an array of one token, right? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121122/new/ https://reviews.llvm.org/D121122 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits