Skip to content

Commit bc25e51

Browse files
authored
Merge pull request #8997 from The-OpenROAD-Project-staging/gpl-data-race
gpl: fix data race in updateDbGcells()
2 parents 4198d3b + bcd7e58 commit bc25e51

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/gpl/src/nesterovBase.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1452,10 +1452,8 @@ void NesterovBaseCommon::updateDbGCells()
14521452
if (db_cbk_) {
14531453
db_cbk_->removeOwner();
14541454
}
1455-
assert(omp_get_thread_num() == 0);
1456-
#pragma omp parallel for num_threads(num_threads_)
1457-
for (auto it = getGCells().begin(); it < getGCells().end(); ++it) {
1458-
auto& gCell = *it; // old-style loop for old OpenMP
1455+
1456+
for (auto& gCell : getGCells()) {
14591457
if (gCell->isInstance()) {
14601458
for (Instance* inst : gCell->insts()) {
14611459
odb::dbInst* db_inst = inst->dbInst();
@@ -1468,6 +1466,7 @@ void NesterovBaseCommon::updateDbGCells()
14681466
}
14691467
}
14701468
}
1469+
14711470
if (db_cbk_) {
14721471
db_cbk_->addOwner(pbc_->db()->getChip()->getBlock());
14731472
}

0 commit comments

Comments
 (0)