================ @@ -670,6 +671,7 @@ llvm::Error ClangDocBitcodeReader::readRecord(unsigned ID, T I) { template <> llvm::Error ClangDocBitcodeReader::readRecord(unsigned ID, Reference *I) { + llvm::TimeTraceScope("clang-doc", "readRecord Reference"); ---------------- ilovepi wrote:
Why `readRecord Reference`? If you want to describe the fact that you're in `readRecord`, just do that, otherwise, if its a logical thing, `Read Record`. If `reference` is important, its fine to leave, but its not clear to me why. https://github.com/llvm/llvm-project/pull/97644 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits