diff options
| author | Clifford Wolf <clifford@clifford.at> | 2014-07-25 12:49:51 +0200 |
|---|---|---|
| committer | Clifford Wolf <clifford@clifford.at> | 2014-07-25 12:49:51 +0200 |
| commit | 0520bfea892291a131134411d587034fcd36bf1c (patch) | |
| tree | 32144c81221d3c70fd833372c40c19eb59b6b319 | |
| parent | c4e4f79a2a2fd5530fa2677245f9361c7b04df70 (diff) | |
Fixed memory corruption in "opt_reduce" pass
| -rw-r--r-- | passes/opt/opt_reduce.cc | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/passes/opt/opt_reduce.cc b/passes/opt/opt_reduce.cc index 913855f4..0cc16ee6 100644 --- a/passes/opt/opt_reduce.cc +++ b/passes/opt/opt_reduce.cc @@ -312,12 +312,14 @@ struct OptReduceWorker // merge identical inputs on $mux and $pmux cells - for (auto &cell_it : module->cells) - { - RTLIL::Cell *cell = cell_it.second; - if ((cell->type != "$mux" && cell->type != "$pmux" && cell->type != "$safe_pmux") || !design->selected(module, cell)) - continue; + std::vector<RTLIL::Cell*> cells; + for (auto &it : module->cells) + if ((it.second->type == "$mux" || it.second->type == "$pmux" || it.second->type == "$safe_pmux") && design->selected(module, it.second)) + cells.push_back(it.second); + + for (auto cell : cells) + { // this optimization is to aggressive for most coarse-grain applications. // but we always want it for multiplexers driving write enable ports. if (do_fine || mem_wren_sigs.check_any(assign_map(cell->connections.at("\\Y")))) |