From 2f3e113db04f505380818b84ca72223accae0f28 Mon Sep 17 00:00:00 2001
From: Matthias Koefferlein <matthias@koefferlein.de>
Date: Sat, 17 Jul 2021 22:06:06 +0200
Subject: [PATCH] Added missing subcircuit mismatch events to netlist compare

---
 src/db/db/dbNetlistCompare.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/db/db/dbNetlistCompare.cc b/src/db/db/dbNetlistCompare.cc
index 2bd44b72b..ae5a3a32b 100644
--- a/src/db/db/dbNetlistCompare.cc
+++ b/src/db/db/dbNetlistCompare.cc
@@ -4243,12 +4243,12 @@ NetlistComparer::do_subcircuit_assignment (const db::Circuit *c1, const db::NetG
     bool mapped = true, valid = true;
     std::vector<std::pair<size_t, size_t> > k = compute_subcircuit_key_for_this (*sc, g1, &c12_circuit_and_pin_mapping, &circuit_pin_mapper, mapped, valid);
 
-    if (! mapped) {
+    if (! mapped || ! valid) {
       if (mp_logger) {
         mp_logger->subcircuit_mismatch (sc.operator-> (), 0);
       }
       good = false;
-    } else if (valid) {
+    } else {
       //  TODO: report devices which cannot be distinguished topologically?
       subcircuit_map.insert (std::make_pair (k, std::make_pair (sc.operator-> (), sc_cat)));
     }