Skip to content

Commit

Permalink
opt_merge: const auto& where suitable
Browse files Browse the repository at this point in the history
  • Loading branch information
@povik @widlarizer
povik authored and widlarizer committed Jul 29, 2024
1 parent 0de4db3 commit 379c462
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions passes/opt/opt_merge.cc
View file
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ struct OptMergeWorker
}
conn_hash = mkhash_xorshift64(a.hash()) + acc;

Check failure on line 106 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, clang) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 106 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, clang-14) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 106 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, gcc) 

‘mkhash_xorshift64’ was not declared in this scope; did you mean ‘Yosys::hashlib::mkhash_xorshift64’?

Check failure on line 106 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-22.04, clang-11) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 106 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, gcc-10) 

‘mkhash_xorshift64’ was not declared in this scope; did you mean ‘Yosys::hashlib::mkhash_xorshift64’?

Check failure on line 106 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (macos-13, clang) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?
} else {
for (auto conn : cell->connections())
for (const auto &conn : cell->connections())
if (!cell->output(conn.first))
conn_hash += mkhash_xorshift64(mkhash(conn.first.hash(), assign_map(conn.second).hash()));

Check failure on line 110 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, clang) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 110 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, clang-14) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 110 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, gcc) 

‘mkhash_xorshift64’ was not declared in this scope; did you mean ‘Yosys::hashlib::mkhash_xorshift64’?

Check failure on line 110 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-22.04, clang-11) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 110 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, gcc-10) 

‘mkhash_xorshift64’ was not declared in this scope; did you mean ‘Yosys::hashlib::mkhash_xorshift64’?

Check failure on line 110 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (macos-13, clang) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Expand All @@ -114,7 +114,7 @@ struct OptMergeWorker
}

uint64_t param_hash = 0;
for (auto &it : cell->parameters)
for (const auto &it : cell->parameters)
param_hash += mkhash_xorshift64(mkhash(it.first.hash(), it.second.hash()));

Check failure on line 118 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, clang) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 118 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, clang-14) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 118 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, gcc) 

‘mkhash_xorshift64’ was not declared in this scope; did you mean ‘Yosys::hashlib::mkhash_xorshift64’?

Check failure on line 118 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-22.04, clang-11) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 118 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, gcc-10) 

‘mkhash_xorshift64’ was not declared in this scope; did you mean ‘Yosys::hashlib::mkhash_xorshift64’?

Check failure on line 118 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (macos-13, clang) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

return conn_hash + mkhash_xorshift64(param_hash + cell->type.hash());

Check failure on line 120 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, clang) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 120 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, clang-14) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 120 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, gcc) 

‘mkhash_xorshift64’ was not declared in this scope; did you mean ‘Yosys::hashlib::mkhash_xorshift64’?

Check failure on line 120 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-22.04, clang-11) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?

Check failure on line 120 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (ubuntu-latest, gcc-10) 

‘mkhash_xorshift64’ was not declared in this scope; did you mean ‘Yosys::hashlib::mkhash_xorshift64’?

Check failure on line 120 in passes/opt/opt_merge.cc

View workflow job for this annotation

GitHub Actions / test-compile (macos-13, clang) 

use of undeclared identifier 'mkhash_xorshift64'; did you mean 'mkhash_xorshift'?
Expand Down

0 comments on commit 379c462

Please sign in to comment.