[PATCH] D46602: [clang-tidy] Store checks profiling info as YAML files

Alexander Kornienko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 9 07:34:06 PDT 2018


alexfh added a comment.

Roman, it looks to me that a simpler storage scheme would be sufficient. For example, YYYYMMDDhhmmss-InputFileName.cpp.csv. Main things are: 1. include a timestamp, so there's no need to overwrite old results, 2. include just the name of the file without any parent directories, 3. put all outputs into the same directory. This way we wouldn't have to create a directory structure and think about stripping a certain prefix (btw, utilities like patch just specify the number of path components to remove from the start, not the actual substring). WDYT?


Repository:
  rL LLVM

https://reviews.llvm.org/D46602





More information about the llvm-commits mailing list