replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[llvm-remark-size-diff] Don't use enum name as auto variable name 

This was confusing the clang-cmake-x86_64-avx2-linux buildbot (gcc version 5.4.0).
This commit is contained in:
Simon Pilgrim 2022-02-05 19:30:03 +00:00
parent 764cd491b1
commit eb9ac2cc14
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/tools/llvm-remark-size-diff/RemarkSizeDiff.cpp
View File

@ -135,13 +135,13 @@ struct DiffsCategorizedByFilesPresent {
static void printFunctionDiff(const FunctionDiff &FD, llvm::raw_ostream &OS) { static void printFunctionDiff(const FunctionDiff &FD, llvm::raw_ostream &OS) {
// Describe which files the function had remarks in. // Describe which files the function had remarks in.
auto FilesPresent = FD.getFilesPresent(); FilesPresent FP = FD.getFilesPresent();
const auto &FuncName = FD.FuncName; const std::string &FuncName = FD.FuncName;
const int64_t InstDiff = FD.getInstDiff(); const int64_t InstDiff = FD.getInstDiff();
assert(InstDiff && "Shouldn't get functions with no size change?"); assert(InstDiff && "Shouldn't get functions with no size change?");
const int64_t StackDiff = FD.getStackDiff(); const int64_t StackDiff = FD.getStackDiff();
// Output an indicator denoting which files the function was present in. // Output an indicator denoting which files the function was present in.
switch (FilesPresent) { switch (FP) {
case FilesPresent::A: case FilesPresent::A:
OS << "-- "; OS << "-- ";
break; break;