[llvm] [tools][llc] Add `--save-stats` option (PR #163967)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 28 02:46:10 PDT 2025
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff origin/main HEAD --extensions cpp -- llvm/tools/llc/llc.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/tools/llc/llc.cpp b/llvm/tools/llc/llc.cpp
index c3fabc7cc..259f07ab3 100644
--- a/llvm/tools/llc/llc.cpp
+++ b/llvm/tools/llc/llc.cpp
@@ -211,11 +211,7 @@ static cl::opt<std::string> RemarksFormat(
cl::desc("The format used for serializing remarks (default: YAML)"),
cl::value_desc("format"), cl::init("yaml"));
-enum SaveStatsMode {
- None,
- Cwd,
- Obj
-};
+enum SaveStatsMode { None, Cwd, Obj };
static cl::opt<SaveStatsMode> SaveStats(
"save-stats",
@@ -224,8 +220,7 @@ static cl::opt<SaveStatsMode> SaveStats(
"file (`-save-stats=obj`). (default: cwd)"),
cl::values(clEnumValN(SaveStatsMode::Cwd, "cwd",
"Save to the current working directory"),
- clEnumValN(SaveStatsMode::Cwd, "",
- ""),
+ clEnumValN(SaveStatsMode::Cwd, "", ""),
clEnumValN(SaveStatsMode::Obj, "obj",
"Save to the output file directory")),
cl::init(SaveStatsMode::None), cl::ValueOptional);
@@ -396,14 +391,14 @@ static int MaybeSaveStats(std::string &&OutputFilename) {
if (SaveStats == SaveStatsMode::None) {
return 0;
}
-
+
SmallString<128> StatsFilename;
if (SaveStats == SaveStatsMode::Obj) {
StatsFilename = OutputFilename;
llvm::sys::path::remove_filename(StatsFilename);
} else {
assert((SaveStats == SaveStatsMode::Cwd) &&
- "Should have been a valid --save-stats value");
+ "Should have been a valid --save-stats value");
}
auto BaseName = llvm::sys::path::filename(OutputFilename);
@@ -419,7 +414,7 @@ static int MaybeSaveStats(std::string &&OutputFilename) {
<< "Unable to open statistics file: " << EC.message() << "\n";
return 1;
}
-
+
llvm::PrintStatisticsJSON(*StatsOS);
return 0;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/163967
More information about the llvm-commits
mailing list