Skip to content

Commit 599e0da

Browse files
committed
[BOLT][NFC] Rename icf-dfs option variable to ICFUseDFS
Rename to avoid collision with profile-use-dfs. Differential Revision: https://reviews.llvm.org/D155513
1 parent 5c106f7 commit 599e0da

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

bolt/lib/Passes/IdenticalCodeFolding.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,9 @@ namespace opts {
3232

3333
extern cl::OptionCategory BoltOptCategory;
3434

35-
static cl::opt<bool> UseDFS("icf-dfs",
36-
cl::desc("use DFS ordering when using -icf option"),
37-
cl::ReallyHidden, cl::cat(BoltOptCategory));
35+
static cl::opt<bool>
36+
ICFUseDFS("icf-dfs", cl::desc("use DFS ordering when using -icf option"),
37+
cl::ReallyHidden, cl::cat(BoltOptCategory));
3838

3939
static cl::opt<bool>
4040
TimeICF("time-icf",
@@ -170,7 +170,7 @@ static bool isIdenticalWith(const BinaryFunction &A, const BinaryFunction &B,
170170
// Process both functions in either DFS or existing order.
171171
SmallVector<const BinaryBasicBlock *, 0> OrderA;
172172
SmallVector<const BinaryBasicBlock *, 0> OrderB;
173-
if (opts::UseDFS) {
173+
if (opts::ICFUseDFS) {
174174
copy(A.dfs(), std::back_inserter(OrderA));
175175
copy(B.dfs(), std::back_inserter(OrderB));
176176
} else {
@@ -360,7 +360,7 @@ void IdenticalCodeFolding::runOnFunctions(BinaryContext &BC) {
360360

361361
// Pre-compute hash before pushing into hashtable.
362362
// Hash instruction operands to minimize hash collisions.
363-
BF.computeHash(opts::UseDFS, [&BC](const MCOperand &Op) {
363+
BF.computeHash(opts::ICFUseDFS, [&BC](const MCOperand &Op) {
364364
return hashInstOperand(BC, Op);
365365
});
366366
};

0 commit comments

Comments
 (0)