Skip to content

Commit 4b75e62

Browse files
committed
reduce changes
Created using spr 1.3.4
2 parents bf44334 + 619e809 commit 4b75e62

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

bolt/lib/Profile/DataAggregator.cpp

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1212,8 +1212,8 @@ std::error_code DataAggregator::parseAggregatedLBREntry() {
12121212
ErrorOr<StringRef> TypeOrErr = parseString(FieldSeparator);
12131213
if (std::error_code EC = TypeOrErr.getError())
12141214
return EC;
1215-
enum TType { TRACE, BRANCH, FT, FT_EXTERNAL_ORIGIN, INVALID };
1216-
auto Type = StringSwitch<TType>(TypeOrErr.get())
1215+
enum AggregatedLBREntry { TRACE, BRANCH, FT, FT_EXTERNAL_ORIGIN, INVALID };
1216+
auto Type = StringSwitch<AggregatedLBREntry>(TypeOrErr.get())
12171217
.Case("T", TRACE)
12181218
.Case("B", BRANCH)
12191219
.Case("F", FT)
@@ -1237,7 +1237,7 @@ std::error_code DataAggregator::parseAggregatedLBREntry() {
12371237
return EC;
12381238

12391239
ErrorOr<Location> TraceFtEnd = std::error_code();
1240-
if (Type == TRACE) {
1240+
if (Type == AggregatedLBREntry::TRACE) {
12411241
while (checkAndConsumeFS()) {
12421242
}
12431243
TraceFtEnd = parseLocationOrOffset();
@@ -1247,12 +1247,13 @@ std::error_code DataAggregator::parseAggregatedLBREntry() {
12471247

12481248
while (checkAndConsumeFS()) {
12491249
}
1250-
ErrorOr<int64_t> Frequency = parseNumberField(FieldSeparator, Type != BRANCH);
1250+
ErrorOr<int64_t> Frequency =
1251+
parseNumberField(FieldSeparator, Type != AggregatedLBREntry::BRANCH);
12511252
if (std::error_code EC = Frequency.getError())
12521253
return EC;
12531254

12541255
uint64_t Mispreds = 0;
1255-
if (Type == BRANCH) {
1256+
if (Type == AggregatedLBREntry::BRANCH) {
12561257
while (checkAndConsumeFS()) {
12571258
}
12581259
ErrorOr<int64_t> MispredsOrErr = parseNumberField(FieldSeparator, true);

llvm/docs/MLGO.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,8 +161,8 @@ requiring no out of tree build-time dependencies.
161161

162162
- ``InteractiveModelRunner``. This is intended for training scenarios where the
163163
training algorithm drives compilation. This model runner has no special
164-
dependencies, and relies on I/O pipes to communicate with a separate process
165-
- presumably a python training algorithm. We do not envision using this in a
164+
dependencies, and relies on I/O pipes to communicate with a separate process,
165+
presumably a python training algorithm. We do not envision using this in a
166166
production environment.
167167

168168
Note that training leaves it to the training infrastructure to handle

0 commit comments

Comments
 (0)