[PATCH] D97094: [Driver] Print process statistics report on CC_PRINT_PROC_STAT env variable.

[Vlad Vereschaka via Phabricator via cfe-commits]

vvereschaka added inline comments.


================
Comment at: clang/lib/Driver/Driver.cpp:4050
                << ", mem=" << ProcStat->PeakMemory << " Kb\n";
-      }
-      if (!StatReportFile.empty()) {
+      } else {
         // CSV format.
----------------
aganea wrote:
> The previous behavior was printing both the output and the CSV file when specifying `-fproc-stat-report -fproc-stat-report=file.csv`, now it would only emit the CSV, is that intended? @sepavloff what do you think?
>now it would only emit the CSV, is that intended?

yes, I thought in that way. I dоn't think that usage of both options at the same time was specially designed. I don't see a reasonable usage case for that for now actually, but I could be wrong. @sepavloff ?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97094/new/

https://reviews.llvm.org/D97094