Skip to content

Commit e3a207b

Browse files
committed
ram: avoid using fmt::join in power/ground pin name error messages
Signed-off-by: Thinh Nguyen <nguyenthinh19011@gmail.com>
1 parent baf136e commit e3a207b

1 file changed

Lines changed: 16 additions & 2 deletions

File tree

src/ram/src/ram.cpp

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -591,17 +591,31 @@ std::map<PortRole, std::string> RamGen::buildPortMap(dbMaster* master)
591591
}
592592

593593
if (power_pin_names_.size() > 1) {
594+
std::string names;
595+
for (const auto& name : power_pin_names_) {
596+
if (!names.empty()) {
597+
names += ", ";
598+
}
599+
names += name;
600+
}
594601
logger_->error(RAM,
595602
42,
596603
"Multiple primary power pin names detected across cells: {}",
597-
fmt::join(power_pin_names_, ", "));
604+
names);
598605
}
599606
if (ground_pin_names_.size() > 1) {
607+
std::string names;
608+
for (const auto& name : ground_pin_names_) {
609+
if (!names.empty()) {
610+
names += ", ";
611+
}
612+
names += name;
613+
}
600614
logger_->error(
601615
RAM,
602616
43,
603617
"Multiple primary ground pin names detected across cells: {}",
604-
fmt::join(ground_pin_names_, ", "));
618+
names);
605619
}
606620

607621
return pin_map;

0 commit comments

Comments
 (0)