================
@@ -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

Reply via email to