Skip to content

Commit c136e72

Browse files
committed
[Remarks] Fix '-fpermissive'. NFC
1 parent b98b567 commit c136e72

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

llvm/tools/llvm-remarkutil/RemarkCounter.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ Error ArgumentCounter::getAllMatchingArgumentsInRemark(
167167

168168
std::optional<std::string> Counter::getGroupByKey(const Remark &Remark) {
169169

170-
switch (GroupBy) {
170+
switch (_GroupBy) {
171171
case GroupBy::PER_FUNCTION:
172172
return Remark.FunctionName.str();
173173
case GroupBy::TOTAL:
@@ -177,7 +177,7 @@ std::optional<std::string> Counter::getGroupByKey(const Remark &Remark) {
177177
if (!Remark.Loc.has_value())
178178
return std::nullopt;
179179

180-
if (GroupBy == GroupBy::PER_FUNCTION_WITH_DEBUG_LOC)
180+
if (_GroupBy == GroupBy::PER_FUNCTION_WITH_DEBUG_LOC)
181181
return Remark.Loc->SourceFilePath.str() + ":" + Remark.FunctionName.str();
182182
return Remark.Loc->SourceFilePath.str();
183183
}
@@ -213,7 +213,7 @@ Error ArgumentCounter::print(StringRef OutputFileName) {
213213
return MaybeOF.takeError();
214214

215215
auto OF = std::move(*MaybeOF);
216-
OF->os() << groupByToStr(GroupBy) << ",";
216+
OF->os() << groupByToStr(_GroupBy) << ",";
217217
unsigned Idx = 0;
218218
for (auto [Key, _] : ArgumentSetIdxMap) {
219219
OF->os() << Key;
@@ -243,7 +243,7 @@ Error RemarkCounter::print(StringRef OutputFileName) {
243243
return MaybeOF.takeError();
244244

245245
auto OF = std::move(*MaybeOF);
246-
OF->os() << groupByToStr(GroupBy) << ","
246+
OF->os() << groupByToStr(_GroupBy) << ","
247247
<< "Count\n";
248248
for (auto [Key, Count] : CountedByRemarksMap)
249249
OF->os() << Key << "," << Count << "\n";

llvm/tools/llvm-remarkutil/RemarkCounter.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -110,9 +110,9 @@ inline Error checkRegex(const Regex &Regex) {
110110
/// Abstract counter class used to define the general required methods for
111111
/// counting a remark.
112112
struct Counter {
113-
GroupBy GroupBy;
113+
GroupBy _GroupBy;
114114
Counter(){};
115-
Counter(enum GroupBy GroupBy) : GroupBy(GroupBy) {}
115+
Counter(enum GroupBy GroupBy) : _GroupBy(GroupBy) {}
116116
/// Obtain the field for collecting remark info based on how we are
117117
/// collecting. Remarks are grouped by FunctionName, Source, Source and
118118
/// Function or collect by file.
@@ -161,7 +161,7 @@ struct ArgumentCounter : Counter {
161161
createArgumentCounter(enum GroupBy GroupBy, ArrayRef<FilterMatcher> Arguments,
162162
StringRef Buffer, Filters &Filter) {
163163
ArgumentCounter AC;
164-
AC.GroupBy = GroupBy;
164+
AC._GroupBy = GroupBy;
165165
for (auto &Arg : Arguments) {
166166
if (Arg.IsRegex) {
167167
if (auto E = checkRegex(Arg.FilterRE))

0 commit comments

Comments
 (0)