1 files changed, 3 insertions, 23 deletions
diff --git a/ice40/arch.cc b/ice40/arch.cc
index 30eea776..74353fe6 100644
--- a/ice40/arch.cc
+++ b/ice40/arch.cc
@@ -242,11 +242,6 @@ IdString Arch::archArgsToId(ArchArgs args) const
 BelId Arch::getBelByName(IdString name) const
 {
     boost::lock_guard<boost::shared_mutex> lock(mtx_);
-    return getBelByNameUnlocked(name);
-}
-
-BelId Arch::getBelByNameUnlocked(IdString name) const
-{
     BelId ret;
 
     if (bel_by_name.empty()) {
@@ -281,13 +276,8 @@ BelRange Arch::getBelsAtSameTile(BelId bel) const
 
 WireId Arch::getWireBelPin(BelId bel, PortPin pin) const
 {
-    boost::shared_lock_guard<boost::shared_mutex> lock(mtx_);
-    return getWireBelPinUnlocked(bel, pin);
-}
-
-WireId Arch::getWireBelPinUnlocked(BelId bel, PortPin pin) const
-{
     WireId ret;
+    boost::shared_lock_guard<boost::shared_mutex> lock(mtx_);
 
     NPNR_ASSERT(bel != BelId());
 
@@ -307,13 +297,8 @@ WireId Arch::getWireBelPinUnlocked(BelId bel, PortPin pin) const
 
 WireId Arch::getWireByName(IdString name) const
 {
-    boost::shared_lock_guard<boost::shared_mutex> lock(mtx_);
-    return getWireByNameUnlocked(name);
-}
-
-WireId Arch::getWireByNameUnlocked(IdString name) const
-{
     WireId ret;
+    boost::shared_lock_guard<boost::shared_mutex> lock(mtx_);
 
     if (wire_by_name.empty()) {
         for (int i = 0; i < chip_info->num_wires; i++)
@@ -331,13 +316,8 @@ WireId Arch::getWireByNameUnlocked(IdString name) const
 
 PipId Arch::getPipByName(IdString name) const
 {
-    boost::shared_lock_guard<boost::shared_mutex> lock(mtx_);
-    return getPipByNameUnlocked(name);
-}
-
-PipId Arch::getPipByNameUnlocked(IdString name) const
-{
     PipId ret;
+    boost::shared_lock_guard<boost::shared_mutex> lock(mtx_);
 
     if (pip_by_name.empty()) {
         for (int i = 0; i < chip_info->num_pips; i++) {